Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

[FxA] The "Sign In" promo is wrongly displayed on a non-owner shot if it's opened on the Firefox profile where you are already logged #5074

Closed
SoftVision-CosminMuntean opened this issue Oct 26, 2018 · 4 comments
Assignees
Labels
[QA]:Major issue Label for QA to mark major issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues

Comments

@SoftVision-CosminMuntean

[Affected versions]:

  • Nightly 65.0a1
  • Screenshots 34.0.0 dev

[Affected Platforms]:

  • All Windows
  • All Mac
  • All Linux

[Prerequisites]:

  • Have a Screenshots FxA account with at least one saved shot.
  • Have a profile where you are already logged in Screenshots FxA account.

[Steps to reproduce]:

  1. Open the Firefox browser with the profile from prerequisites.
  2. Open a saved shot created on a different profile (eg: this link).
  3. Observe the shot page.

[Expected result]:

  • The "Sign In" promo is not displayed.

[Actual results]:

  • The "Sign In" promo is displayed even if you are already logged in.

[ Notes]:

  • Here is a screen recording of the issue:
    sign in promo on non owner
@SoftVision-CosminMuntean SoftVision-CosminMuntean added the [QA]:Major issue Label for QA to mark major issues logged label Oct 26, 2018
punamdahiya added a commit to punamdahiya/screenshots that referenced this issue Oct 26, 2018
@punamdahiya punamdahiya self-assigned this Oct 26, 2018
@punamdahiya
Copy link
Contributor

Attached PR with the fix

@ianb ianb added this to the Sprint 25 (65-2) 🎃 milestone Oct 29, 2018
punamdahiya added a commit that referenced this issue Oct 29, 2018
Fixes #5074 - Hide Signin banner on non-owned shot if authenticated
@SoftVision-CosminMuntean
Copy link
Author

I have verified this issue with latest Screenshots (34.0.0) dev version and the issue is no longer reproducible on a received shot.
But, I have noticed that the "Sign In" promo is still displayed if you navigate to a non-owner shot that has been deleted or has expired.

[Prerequisities]:

  • The latest Screenshots dev version is installed on a Firefox profile.
  • You are already logged with a valid FxA account.

[Steps to reproduce]:

  1. Open the Firefox browser with the profile from prerequisites.
  2. Navigate to an expired shot or deleted shot that was made on another profile (eg: this one).
  3. Observe the page.

[Expected result]:

  • The "Sign In" promo is not displayed.

[Actual results]:

  • The "Sign In" promo is displayed even if you are already logged in.

Here is a screen recording of the issue:
sign in probo

@punamdahiya
Copy link
Contributor

@SoftVision-CosminMuntean Thanks for catching this, we should pass accountId cookie info (hasFxa) in not found pages https://github.com/mozilla-services/screenshots/blob/master/server/src/pages/not-found/view.js#L32, will submit the PR

@SoftVision-CosminMuntean
Copy link
Author

I have verified this issue using the latest Screenshots (34.0.0) dev version and is no longer reproducible. Tested on Windows 7 x64, Mac 10.13 and Arch Linux 4.12.

@SoftVision-CosminMuntean SoftVision-CosminMuntean added the [QA]:Verified fixed Label for QA to mark verified fixed issues label Nov 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
[QA]:Major issue Label for QA to mark major issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues
Projects
None yet
Development

No branches or pull requests

3 participants