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

Testing website security: Google permissions, cookies and HTTPS #25

Open
storytracer opened this issue May 19, 2021 · 1 comment
Open

Comments

@storytracer
Copy link

First of all, thank you so much for this amazing tool. I only discovered it today and am absolutely stunned by your great work. However, when I tweeted about it, there was immediate concern about the security of the testing website (see reactions to tweet).

For users wanting to test the tool it would be great if you could clarify/fix the following points on the testing website:

  • Force HTTPS for all website visitors
  • Properly name and approve the Google API OAuth application which is asking for permission to users' accounts (it is currently un-approved by Google and shows a warning to users)
  • Clarify what Google account permissions will be requested and why
  • Explain why Google cookies are mandatory for the use of the tool
@adammertel
Copy link
Owner

adammertel commented May 23, 2021

Hi @canbuffi thank you very much for reaching me here.
(I am sorry I was not able to respond on Twitter, I am not very active there :/)

I was trying to rewrite the website to use HTTPS only previously but the external services stopped working. But I should give it at least one more shot, there is definitely a workaround.

With the rest of your comments, I absolutely agree. We have been using Historical Geocoding Assistant mainly internally so the problem with the security was not that significant. I should find some time in the next couple of weeks to fix them - thanks for the tips.

If you have other ideas on how to make this tool more convenient, and secure for external users, please let me know, I am happy to implement them :)

Have a nice rest of the weekend and thank you once again for your notes

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

No branches or pull requests

2 participants