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

Click on Twitter favicon in tipping banner redirects to an error page #12214

Closed
GeetaSarvadnya opened this issue Oct 20, 2020 · 6 comments · Fixed by brave/brave-core#6918
Closed

Comments

@GeetaSarvadnya
Copy link

Description

Click on twitter favicon in tipping banner redirects to an error page

Steps to Reproduce

  1. Clean profile 1.17.x
  2. Enable rewards
  3. Open https://twitter.com/jonathansampson
  4. Open BR panel and click on Send tip
  5. Click on Twitter favicon in tipping banner

Actual result:

Click on twitter favicon in tipping banner redirects to an error page
image
image

Expected result:

should not show any error message

Reproduces how often:

Always

Brave version (brave://version info)

Brave 1.17.42 Chromium: 86.0.4240.99 (Official Build) nightly (64-bit)
Revision 002668237e13d38aabd6d11c2d216dd22b736ff2-refs/branch-heads/4240@{#1229}
OS Windows 10 OS Version 1903 (Build 18362.1082)

Version/Channel Information:

  • Can you reproduce this issue with the current release? No
  • Can you reproduce this issue with the beta channel? No
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? NA
  • Does the issue resolve itself when disabling Brave Rewards? NA
  • Is the issue reproducible on the latest version of Chrome? NA

Miscellaneous Information:

cc: @brave/legacy_qa @rebron @NejcZdovc

@NejcZdovc
Copy link
Contributor

this is not regression as the same thing happens for me on previous versions

@GeetaSarvadnya
Copy link
Author

@NejcZdovc I am getting an empty screen in Release channel 1.15.x

@NejcZdovc
Copy link
Contributor

yes I get empty screen as well. Problem is that link that we get from server doesn't have https in front and because of that it will not open correctly. Checking with publisher team what to do

@zenparsing
Copy link

It seems reasonable to not display a social media link if the link is not a valid absolute URL. What do you think @NejcZdovc?

@NejcZdovc
Copy link
Contributor

@zenparsing that would work yup. We should also fix it on publisher side

@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Nov 23, 2020

Verification passed on


Brave | 1.18.59 Chromium: 87.0.4280.67 (Official Build) dev (64-bit)
-- | --
Revision | 0e5d92df40086cf0050c00f87b11da1b14580930-refs/branch-heads/4280@{#1441}
OS | Windows 10 OS Version 2004 (Build 19041.630)

image


Verified passed with

Brave	1.18.62 Chromium: 87.0.4280.67 (Official Build) dev (x86_64)
Revision	0e5d92df40086cf0050c00f87b11da1b14580930-refs/branch-heads/4280@{#1441}
OS	macOS Version 10.14.6 (Build 18G6042)

Verified STR from description

Reproduced the issue using 1.17.x

Screen Shot 2020-11-25 at 11 06 08 AM

Screen Shot 2020-11-25 at 11 06 18 AM

Confirmed visiting same publisher on 1.18.x did not show the favicons.

Screen Shot 2020-11-25 at 11 05 22 AM

Confirmed visiting a publisher with valid URLs in their favicons did show the favicons. Clicking on the favicon directed me to the expected URL.

Screen Shot 2020-11-25 at 11 27 50 AM

Screen Shot 2020-11-25 at 11 28 04 AM


Verification passed on

Brave 1.18.62 Chromium: 87.0.4280.67 (Official Build) dev (64-bit)
Revision 0e5d92df40086cf0050c00f87b11da1b14580930-refs/branch-heads/4280@{#1441}
OS Ubuntu 18.04 LTS

@rebron rebron changed the title Click on twitter favicon in tipping banner redirects to an error page Click on Twitter favicon in tipping banner redirects to an error page Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment