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

Plugin logs in to live version while settings are set to acceptatie or a bogus address #20

Closed
tevas8 opened this issue Jul 3, 2017 · 1 comment

Comments

@tevas8
Copy link

tevas8 commented Jul 3, 2017

Version

2.0

Description

The user can change the server address in the options menu and save it. Despite the address being correctly saved to a random address, if the user tries a valid user+pass combination for the live server, the plugin will accept it and log in to the live server regardless of the address set in the options menu.

This happens in both FF and Chrome.

While playing around with the server address setting it to non-existent ones, the bug stopped from occuring and was not possible to reproduce.

When making a clean install of FF and the plugin, the bug is reproduceable.

Actual behaviour

When logging in with a combination of username and password that is valid for the live server, the plugin will log the user in even when the server has been changed to something other than https://partner.voipgrid.nl.

Expected behaviour

It should not be possible to log in to the live server after changing the address in the options menu to anything other than https://partner.voipgrid.nl

Steps to reproduce

  1. Make sure you are not logged in the plugin
  2. Open the options menu for the plugin ("about:addons" in Firefox, or "chrome://extensions" in Chrome)
  3. Set the address to https://ANYTHING.voipgrid.nl and save it
  4. Log in using a combination of user+pass that is valid for https://partner.voipgrid.nl
  5. Open the options menu and notice how the address has been changed to https://partner.voipgrid.nl
  6. Click on Settings and notice how it redirects to https://partner.voipgrid.nl

Attached video

FF login to wrong server.zip

@tevas8 tevas8 added the bug label Jul 3, 2017
@ghost ghost added this to In Progress in click-to-dial v2 Jul 3, 2017
@ghost ghost assigned ghost and TomOffringa Jul 3, 2017
@TomOffringa TomOffringa removed their assignment Jul 3, 2017
@ghost ghost assigned TomOffringa and unassigned ghost Jul 3, 2017
@TomOffringa
Copy link
Contributor

Fixed with ce9eaf3

@TomOffringa TomOffringa moved this from In Progress to Done in click-to-dial v2 Jul 3, 2017
@TomOffringa TomOffringa moved this from In Progress to Done in click-to-dial v2 Jul 5, 2017
@ghost ghost unassigned TomOffringa Jul 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Development

No branches or pull requests

2 participants