Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can not edit memos in mobile version #187

Closed
pythonius21 opened this issue Jun 17, 2024 · 22 comments
Closed

Can not edit memos in mobile version #187

pythonius21 opened this issue Jun 17, 2024 · 22 comments

Comments

@pythonius21
Copy link

I can edit the memos via web but not in the android version (fdroid and Google play version tested), change memos status to public,workspace and private, still can't edit via Android app.

It gives me an blank error popup
Screenshot_20240617-081056

@pythonius21
Copy link
Author

version 0.8.1
Which I'm currently running

@pythonius21
Copy link
Author

And no error messages on the server side.
Running latest server version (docker compose), latest android app, tried via token auth and normal, no differences

@mudkipme
Copy link
Owner

mudkipme commented Jun 17, 2024

I'm unable to reproduce this issue with both Memos 0.22.2 and 0.21.0. Could you use a proxy tool to find out the request url & body, and responses happened when you tap the save button?

The access log of your http server and your Memos version may also help.

@pythonius21
Copy link
Author

I use a reverse proxy manager (Nginx Proxy Manager) and the error logs says "Connection reset by peer" while with toke auth and normal login.
The Logcat via android says

06-17 15:12:26.206  2205  2286 D ShellBackPreview: Received backNavigationInfo:BackNavigationInfo{mType=TYPE_CALLBACK (4), mOnBackNavigationDone=android.os.RemoteCallback@21ac1e5, mOnBackInvokedCallback=android.window.IOnBackInvokedCallback$Stub$Proxy@89684ba, mPrepareRemoteAnimation=false, mAnimationCallback=false, mCustomizeAnimationInfo=null}
06-17 15:12:26.241  2305  2361 I deeptouch: I0000 00:00:1718629946.241190    2361 tf_lite_classifier.cc:413] Event stream classified as kNone

It worked before, but after a update from server (maybe) it stopped, I test an older server version and tell my results!

@pythonius21
Copy link
Author

I use a reverse proxy manager (Nginx Proxy Manager) and the error logs says "Connection reset by peer" while with toke auth and normal login. The Logcat via android says

06-17 15:12:26.206  2205  2286 D ShellBackPreview: Received backNavigationInfo:BackNavigationInfo{mType=TYPE_CALLBACK (4), mOnBackNavigationDone=android.os.RemoteCallback@21ac1e5, mOnBackInvokedCallback=android.window.IOnBackInvokedCallback$Stub$Proxy@89684ba, mPrepareRemoteAnimation=false, mAnimationCallback=false, mCustomizeAnimationInfo=null}
06-17 15:12:26.241  2305  2361 I deeptouch: I0000 00:00:1718629946.241190    2361 tf_lite_classifier.cc:413] Event stream classified as kNone

It worked before, but after a update from server (maybe) it stopped, I test an older server version and tell my results!

So I tried now with test server and the error logs says failed (104: Connection reset by peer) while reading response header from upstream, client: *REDACTED*, server: *REDACTED*, request: "PATCH /api/v1/memos/14 HTTP/2.0", upstream: "http://192.168.178.90:5230/api/v1/memos/14", host: "*REDACTED*"

@pythonius21
Copy link
Author

I tried disabling SSO from authelia, used a VPN connection, no chance...

@ganeshlab
Copy link

I had the same issue. I was using the token to login because the default method wasn't working in the beta. Cleared data on the android app and switched back to default login and now everything works as usual.

@pythonius21
Copy link
Author

I had the same issue. I was using the token to login because the default method wasn't working in the beta. Cleared data on the android app and switched back to default login and now everything works as usual.

Doesn't work for me unfortunately, cleared cache, installed older versions, tried token and the default login, still doesn't work.

@pythonius21
Copy link
Author

I will close this issue, as it is apparently only on my end the problem. But still thanks for the help!

@inertia42
Copy link

I will close this issue, as it is apparently only on my end the problem. But still thanks for the help!

I have the same issue. The solution from previous replies doesn't work for me either. Maybe you can reopen this issue.

@pythonius21 pythonius21 reopened this Jun 19, 2024
@cdefine
Copy link

cdefine commented Jun 19, 2024

i have the same problem,i cannot edit or archive memos but can create new memo in my andriod device. everything is ok in web page.
recv() failed (104: Connection reset by peer) while reading response header from upstream, client: 192.168.0.106, server: , request: "PATCH /api/v1/memos/59 HTTP/1.1", upstream: "http://192.168.0.111:5230/api/v1/memos/59", host: "192.168.0.105:8088"

@mudkipme
Copy link
Owner

mudkipme commented Jun 19, 2024

"Connection reset by peer" looks like a server panic when handling request. Is there any logs on Memos' side? Do you use any architecture other than amd64? Did the Memos process or container restarted?

And I'd like to know whether using 0.22.1, 0.22.2 and stable Docker tag could make a different.

You may also try connecting to Memos server directly without a reverse proxy and see if there's any logs.

@mudkipme mudkipme added the question Further information is requested label Jun 19, 2024
@pythonius21
Copy link
Author

"Connection reset by peer" looks like a server panic when handling request. Is there any logs on Memos' side? Do you use any architecture other than amd64? Did the Memos process or container restarted?

And I'd like to know whether using 0.22.1, 0.22.2 and stable Docker tag could make a different.

You may also try connecting to Memos server directly without a reverse proxy and see if there's any logs.

I can confirm that connecting directly to it does work!

@mudkipme
Copy link
Owner

"Connection reset by peer" looks like a server panic when handling request. Is there any logs on Memos' side? Do you use any architecture other than amd64? Did the Memos process or container restarted?
And I'd like to know whether using 0.22.1, 0.22.2 and stable Docker tag could make a different.
You may also try connecting to Memos server directly without a reverse proxy and see if there's any logs.

I can confirm that connecting directly to it does work!

Thanks for the information!

May I see your nginx configuration and nginx version to help reproduce this issue?

@pythonius21
Copy link
Author

pythonius21 commented Jun 19, 2024

"Connection reset by peer" looks like a server panic when handling request. Is there any logs on Memos' side? Do you use any architecture other than amd64? Did the Memos process or container restarted?
And I'd like to know whether using 0.22.1, 0.22.2 and stable Docker tag could make a different.
You may also try connecting to Memos server directly without a reverse proxy and see if there's any logs.

I can confirm that connecting directly to it does work!

Thanks for the information!

May I see your nginx configuration and nginx version to help reproduce this issue?

As I stated earlier I'm using Nginx Proxy Manager version 2.11.2
but here my configurations:
librewolf_VQFi8itLFs
librewolf_EqdFpJrXVo
librewolf_VIKIXuFo5p

@inertia42
Copy link

My memos instance is running on fly.io, and I can't modify the proxy settings of the instance. When I modify a memo, the server logs also show

proxy nrt [error] [PU02] could not complete HTTP request to instance: connection error: Connection reset by peer (os error 104)

I don't quite understand the source of this issue. Is it a server-side problem, an upstream memos issue, or an Android client problem? Thank you!

@mudkipme
Copy link
Owner

mudkipme commented Jun 20, 2024

I can finally reproduce this issue with Nginx Proxy Manager and curl command to the PATCH request. Further investigation shows the request didn’t reach the handler method of Memos server so there might be something wrong with the echo library or mixing HTTP/GRPC methods.

I’ll try to figure out the exact issue and submit a pull request to Memos if there can be a fix. Otherwise, Moe Memos will switch to Protobuf requests like the web app for 0.22+.

Update:
I figured out the exact cause and will submit a pull request to Memos soon.

@mudkipme
Copy link
Owner

The fix is merged, please try upgrading the server to neosmemo/memos:latest or 0.22.3 docker tag.

@cdefine
Copy link

cdefine commented Jun 21, 2024

The fix is merged, please try upgrading the server to neosmemo/memos:latest or 0.22.3 docker tag.

thanks for your fix!!!
i upgrade the server to 0.22.3,most of functions are normal ,but cannot upload image now 。the server response 500

@mudkipme
Copy link
Owner

mudkipme commented Jun 21, 2024

The fix is merged, please try upgrading the server to neosmemo/memos:latest or 0.22.3 docker tag.

thanks for your fix!!! i upgrade the server to 0.22.3,most of functions are normal ,but cannot upload image now 。the server response 500

Is there any error log in Memos or the proxy server when uploading images?

@pythonius21
Copy link
Author

The fix is merged, please try upgrading the server to neosmemo/memos:latest or 0.22.3 docker tag.

I can confirm that it is indeed working! Thank you very much :)

@pisceswb
Copy link

cannot upload image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants