This repository has been archived by the owner. It is now read-only.

Dropbox.com - Regression in PM v27.5.0 #1363

Closed
THEtomaso opened this Issue Sep 24, 2017 · 8 comments

Comments

Projects
None yet
4 participants
@THEtomaso

THEtomaso commented Sep 24, 2017

Seems that elements @ Dropbox.com can't be shown anymore in Pale Moon v27.5.0, although the pages rendered just fine in v27.4.2.
Examples:
https://www.dropbox.com/s/n8n2bu2tshtetnb/
https://www.dropbox.com/sh/wackcyek8nzziaf/wJ1hAkF49U/
No elements are shown at all..
Just a white page.

(Pale Moon v27.5.0,x64 / Win8.1Pro)

@THEtomaso THEtomaso changed the title from Dropbox videos - Regression in PM v27.5.0 to Dropbox.com - Regression in PM v27.5.0 Sep 24, 2017

@JustOff

This comment has been minimized.

Show comment
Hide comment
@JustOff

JustOff Sep 24, 2017

Contributor

One more UserAgent/version sniffing issue. Setting general.useragent.override.dropbox.com to Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.9) Gecko/20100101 Firefox/45.9 resolves it.

Contributor

JustOff commented Sep 24, 2017

One more UserAgent/version sniffing issue. Setting general.useragent.override.dropbox.com to Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.9) Gecko/20100101 Firefox/45.9 resolves it.

@Sa-Ja-Di

This comment has been minimized.

Show comment
Hide comment
@Sa-Ja-Di

Sa-Ja-Di Sep 24, 2017

Works with native user agent mode without issues in the latest unstable

Sa-Ja-Di commented Sep 24, 2017

Works with native user agent mode without issues in the latest unstable

@JustOff

This comment has been minimized.

Show comment
Hide comment
@JustOff

JustOff Sep 24, 2017

Contributor

Well, then it's only version sniffing issue 😜

Contributor

JustOff commented Sep 24, 2017

Well, then it's only version sniffing issue 😜

@THEtomaso

This comment has been minimized.

Show comment
Hide comment
@THEtomaso

THEtomaso Sep 24, 2017

Confirmed.
I used the UA string from PM v27.4.2, and the pages work again. :)
You should include this override in the next point release.

--

EDIT:
..and while you're at it, could you add a UA override for this site too?:
https://www.dn.no/
When using Pale Moon, they serve you the mobile version of their page.
I've sent them mails about it a long time ago, but they don't seem to care at all.
It's a popular news page here in Norway.

THEtomaso commented Sep 24, 2017

Confirmed.
I used the UA string from PM v27.4.2, and the pages work again. :)
You should include this override in the next point release.

--

EDIT:
..and while you're at it, could you add a UA override for this site too?:
https://www.dn.no/
When using Pale Moon, they serve you the mobile version of their page.
I've sent them mails about it a long time ago, but they don't seem to care at all.
It's a popular news page here in Norway.

@wolfbeast

This comment has been minimized.

Show comment
Hide comment
@wolfbeast

wolfbeast Sep 24, 2017

Member

Dropbox should be served the native UA in that case. Patches welcome to that effect to the UA override include in browser/branding/shared

Member

wolfbeast commented Sep 24, 2017

Dropbox should be served the native UA in that case. Patches welcome to that effect to the UA override include in browser/branding/shared

@JustOff

This comment has been minimized.

Show comment
Hide comment
@JustOff

JustOff Sep 25, 2017

Contributor

@wolfbeast
Btw, what about new GRE_VERSION? You bumped it twice in a row in milestone.txt, but not in uaoverrides.inc.

Contributor

JustOff commented Sep 25, 2017

@wolfbeast
Btw, what about new GRE_VERSION? You bumped it twice in a row in milestone.txt, but not in uaoverrides.inc.

@wolfbeast

This comment has been minimized.

Show comment
Hide comment
@wolfbeast

wolfbeast Sep 25, 2017

Member

Taken care of. thanks for the reminder.

Member

wolfbeast commented Sep 25, 2017

Taken care of. thanks for the reminder.

@wolfbeast wolfbeast closed this in e4471c5 Sep 25, 2017

wolfbeast added a commit that referenced this issue Sep 25, 2017

@THEtomaso

This comment has been minimized.

Show comment
Hide comment
@THEtomaso

THEtomaso Sep 25, 2017

This UA override works for dn.no (which I posted a link to in my previous post):
Mozilla/5.0 (Windows NT 6.3; rv:45.9) Gecko/20100101 Firefox/45.9
Don't know if it's the "best" string to use though.
Could someone take a look at it, and perhaps make sure that it's included in the next point release?
Sorry if it's "off topic", but it seemed a little overkill to open a new issue just because of this.

THEtomaso commented Sep 25, 2017

This UA override works for dn.no (which I posted a link to in my previous post):
Mozilla/5.0 (Windows NT 6.3; rv:45.9) Gecko/20100101 Firefox/45.9
Don't know if it's the "best" string to use though.
Could someone take a look at it, and perhaps make sure that it's included in the next point release?
Sorry if it's "off topic", but it seemed a little overkill to open a new issue just because of this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.