search

jupyter / try.jupyter.org

Try Jupyter!
https://try.jupyter.org
BSD 3-Clause "New" or "Revised" License
10 stars 13 forks source link

Rely on top-level redirector #18

Closed rgbkrk closed 7 years ago

rgbkrk commented 7 years ago

tmp54 is dead, long live the king

https://github.com/jupyter/help/issues/88

rgbkrk commented 7 years ago

Disabling Travis for this repo, as it's not necessary.

rgbkrk commented 7 years ago

Welp, tmp57 doesn't have the right content security policy set.

Refused to display 'https://tmp57.tmpnb.org/user/lABDPwkTgydM/tree' in a frame because an ancestor violates the following Content Security Policy directive: "frame-ancestors 'self' https://_.jupyter.org https://jupyter.github.io https://_.tmpnb.org".

.____.

minrk commented 7 years ago

@rgbkrk did you fix something manually? it seems to be working for me on all workers, including tmp57.

rgbkrk commented 7 years ago

Nope, haven't changed anything. Only reported back here.

minrk commented 7 years ago

Is it working for you? Safari and Chrome both appear to work from here.

rgbkrk commented 7 years ago

screen shot 2016-10-07 at 9 14 22 am

rgbkrk commented 7 years ago

It does the redirection right, but then when I get to the underlying notebook server the CSP rejects it. Both for tmp56 and tmp57.

minrk commented 7 years ago

weird...

screen shot 2016-10-07 at 15 17 46

Chrome version?

rgbkrk commented 7 years ago

Version 53.0.2785.116 (64-bit)

Looks like an update needs to be done for me.

rgbkrk commented 7 years ago

And I've updated to Version 53.0.2785.143 (64-bit). Still a problem.

minrk commented 7 years ago

I've got 54.0.2840.50 (beta channel), so maybe that's relevant. The setting doesn't appear to be wrong, though.

Can you check in incognito and/or other browsers?

rgbkrk commented 7 years ago

screen shot 2016-10-07 at 9 32 00 am