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

imported settings for save of syncthing (not the forked) don't enable the gui #1189

Closed
pcause opened this issue Dec 3, 2024 · 4 comments
Closed
Assignees
Labels
Milestone

Comments

@pcause
Copy link

pcause commented Dec 3, 2024

Description of the issue

I exported my setting from the syncthing android and uninstalled. i installed this version (the fork), imported the saved settings. I can see all my shared folders but when I try to access the webgui i get a login screen and not the webgui.

I looked at the saved config.xml and there is a syncthing username and a password.

Turns out I had to go into setting and set a webgui password. The saved password wasn't being used.

Since users of the old version are likely to be migrating, if this step is needed, I'd suggest you mention that in the startup screens.

Reproduction Steps

see above.

@Catfriend1
Copy link
Owner

We can put the default pw (api key) in if its empty. But the pw is encrypted in the xml, so we won't know it.

@pcause
Copy link
Author

pcause commented Dec 4, 2024

Makes sense, then I'd suggest you add a warning/explanation in the onboarding screens (where you ask for permissions) telling the user that if they want to use the GUI they need to create a password in the section of the settings.

Also don't import the gui settings part of the saved info, and on import, add a message after the import that the username and gui are bot imported.

@Catfriend1
Copy link
Owner

PR welcome, good idea :-)

@Catfriend1
Copy link
Owner

Ah, I see. The official app does not write sharedpreferences.dat thus exporting and importing between Syncthing-Fork versions works fine here and exporting from the official app to STF does not.

@Catfriend1 Catfriend1 self-assigned this Dec 10, 2024
@Catfriend1 Catfriend1 added this to the v1.28.1 milestone Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants