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

Issue 443, fixed github logout issue #445

Merged
merged 4 commits into from Dec 29, 2023
Merged

Issue 443, fixed github logout issue #445

merged 4 commits into from Dec 29, 2023

Conversation

OliverWang13
Copy link
Member

This should all be dealt with.

On your end, can you check if two things are working:

  1. Go to https://global-privacy-control.glitch.me/, the DOM signal and header should still be being sent.
  2. Inspect the page and look at the cookies, is there a usprivacy string cookie with the value 1NYN? (there should be)
  3. Using the popup, opt back in to sales then reload the page.
  4. The DOM signal and header should not be sent anymore.
  5. The usprivacy string cookie should still be there but with 1NNN
  6. Navigate to https://www.33across.com/
  7. In the cookies, there should be a cookie named 33x_nc with the value 33Across+Optout
  8. If you opt in to that site, that cookie should be deleted
  9. If you opt out of that site again, the cookie should reappear
  10. Finally, try to replicate the github issue again and you should not be logged out

Please try to do these checks for both Chrome and Firefox

@OliverWang13 OliverWang13 added the bug Something isn't working label Dec 18, 2023
@OliverWang13
Copy link
Member Author

CI tests were failing at first but I updated our tests to reflect the changes in the functions I made, as well as added some new test cases as well

@SebastianZimmeck SebastianZimmeck requested review from Mattm27 and removed request for Jocelyn0830 December 26, 2023 16:51
@SebastianZimmeck
Copy link
Member

@Mattm27 will review.

Copy link
Member

@Mattm27 Mattm27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For Chrome, all checks for both https://global-privacy-control.glitch.me/ and https://www.33across.com/ look good and there is no longer a logout issue with Github. For Firefox, on https://global-privacy-control.glitch.me/ after opting into sales and reloading page, DOM signal was still present (check 4) but with yellow thumbs up instead of blue. Other than that, all other checks for Firefox pass including those on https://www.33across.com/ and there is no longer a logout issue with Github.

@OliverWang13
Copy link
Member Author

This looks like it's due to a change on Firefox's end, so I think this is not an issue. Normally, by default, navigator.globalPrivacyControl is undefined, causing the red thumbs down, but it seems as though Firefox has changed their default to be so navigator.globalPrivacyControl is False, causing the yellow thumbs up. This might be something that should be changed with the glitch site, but should not be an issue for us.

@OliverWang13 OliverWang13 merged commit 2d1bd83 into main Dec 29, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants