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

Make separate "Update & Settings" menu item #29

Closed
micahflee opened this issue Apr 10, 2013 · 11 comments
Closed

Make separate "Update & Settings" menu item #29

micahflee opened this issue Apr 10, 2013 · 11 comments

Comments

@micahflee
Copy link
Collaborator

This is related to #25 and #28. Rather than just making a new "Tor Browser Update" .desktop file that uses a -force-check-update flag, the menu item should use a -settings flag, and display a GUI interface that lets you do a couple things:

  • Force an update check
  • Toggle between preferring alpha or stable
  • Launch TBB

This can be where all other settings we might want to add in the future can exist.

If it looks like this can get into deb.torproject.org soon, it might make more sense first to just finish #25 first, and add this later.

@Phoul
Copy link

Phoul commented Apr 10, 2013

It would also be helpful if the user had the ability to pick the Pluggable Transports Browser Bundle.

@micahflee
Copy link
Collaborator Author

Another option should be "Update over Tor".

@micahflee
Copy link
Collaborator Author

I've been working on this. Now if you run "torbrowser-launcher -settings", it looks like this. Though it doesn't actually save the settings yet.

tbl_settings

@adrelanos
Copy link

Looks very well.

@ioerror
Copy link
Contributor

ioerror commented Apr 18, 2013

Will those be checked by default?

@micahflee
Copy link
Collaborator Author

@ioerror, yes they will default to being checked. I've actually changed the text from "Check for and download updates over Tor" to "Download updates over Tor (recommended)".

I'm also considering adding another checkbox to address #41 that says something like "Download over hidden service", and that's only enabled if "Download updates over Tor" is checked. More thoughts on that in the other issue.

@adrelanos
Copy link

Some wishes for storing settings:

  • If the options file has been deleted: use the internal defaults and rewrite that file.
  • If one option is missing or invalid in options file: use the internal defaults.
  • If half of the options is in there (because a new version of torbrowser-launcher introduced new settings): use the existing values, don't overwrite the old ones and add the new ones using internal defaults

@micahflee
Copy link
Collaborator Author

I just pushed a bunch of updates to this bug. There's now a settings dialogue and it has it's own menu item, and it saves your preferences. It's also now possible to choose if you want stable or alpha (though not yet Obfsproxy because of issues discussed in #38).

@adrelanos that's a good idea for choosing how to migrate the settings file. These recent commits include a settings file migration from 0.0.1 to 0.0.2, but I haven't thoroughly tested this migration yet.

I also haven't made it actually download anything over Tor yet, it just saves your setting there. And I also haven't yet added the Tor hidden service checkbox either (#41).

@adrelanos
Copy link

I think I found the holy grail for configuration files. .d-style configuration folders. In bash:

for i in /etc/tor-launcher.d/*; do
   if [ -f "$i" ]; then
      source "$i"
   fi
done

They get sourced in lexical order. Which means, you could have a /etc/tor-launcher.d/10_default and always add new comments and new default settings to that file. The user could use 20_user or 90_user and overrule any defaults you may have set. Dpkg (you) can replace the default configuration file and users can always overrule with what they want, or script/applications depending on your app can drop configurations.

If it's equally simple in Python, I recommend that way.

@legrostdg
Copy link

A more simple format for the configuration file(s) (.ini, yaml, ...) would be a plus, too.

@micahflee
Copy link
Collaborator Author

This issue (making a separate settings dialog) is already implemented and works, so I'm closing it. The issues for updating over Tor is #8 and using a hidden service is #41.

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

5 participants