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

RTC breaks notebook links #202

Open
thesuperzapper opened this issue Oct 13, 2023 · 22 comments · May be fixed by #418
Open

RTC breaks notebook links #202

thesuperzapper opened this issue Oct 13, 2023 · 22 comments · May be fixed by #418
Labels
bug Something isn't working

Comments

@thesuperzapper
Copy link

Description

When you click the "notebook" button, you get a 404:

  • Screenshot 2023-10-12 at 21 05 06

This is because of the extra "RTC:" which is added to the file path (which is not actually there):

  • Screenshot 2023-10-12 at 21 05 59

Reproduce

  1. Ensure pip install notebook is installed
  2. Try clicking the notebook button
  3. See 404
@thesuperzapper thesuperzapper added the bug Something isn't working label Oct 13, 2023
@welcome
Copy link

welcome bot commented Oct 13, 2023

Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗

If you haven't done so already, check out Jupyter's Code of Conduct. Also, please try to follow the issue template as it helps other other community members to contribute more effectively.
welcome
You can meet the other Jovyans by joining our Discourse forum. There is also an intro thread there where you can stop by and say Hi! 👋

Welcome to the Jupyter community! 🎉

@jrouly
Copy link

jrouly commented Nov 17, 2023

This issue also impacts the voila-dashboards extension (issue). The URL seems to prepend the filename with RTC: which looks like it breaks a number of expectations.

@jochym
Copy link

jochym commented Nov 30, 2023

This also affects jupyterlab-latex (jupyterlab/jupyterlab-latex#224) for the same reason. It is even worse - the RTC is present even if the RTC is disabled with --YDocExtension.disable_rtc=True while the extension is enabled (just RTC is disabled).
Please treat it as high priority - it renders multiple use cases impossible.

@jrouly
Copy link

jrouly commented Nov 30, 2023

Based on discussion in #183 I suspect now that the downstream plugins breaking may be considered a feature, not a bug.

However the original issue report here as to a broken feature within JupyterLab itself is pretty damning.

@Podolyakofs
Copy link

Also this affects jupyterlab-h5web#121, same file path error

@frankebersoll
Copy link

This also affects jupyterlab_rise: Slideshows won't work when RTC is enabled because of invalid links, you get 404 all over the place. Same for "File > Save and export notebook as..."

@betolink
Copy link

I ran into the same issue with silx-kit/jupyterlab-h5web#121 Is this a problem with the extensions?

@fcollonval @davidbrochart

@davidbrochart
Copy link
Collaborator

Is this a problem with the extensions?

Yes it probably doesn't support custom drives.

@BwL1289
Copy link

BwL1289 commented Feb 22, 2024

Same with search and replace. I'm opening a ticket there.

@davidbrochart
Copy link
Collaborator

I'm not going to fix every extension, it might be better to open an issue there.

@krassowski
Copy link
Member

krassowski commented Feb 25, 2024

Calling every extension out there broken is a bit of a strong statement. Incompatible sounds more accurate for me. Could JupyterLab (or the collaboration extension) provide a compatibility layer router to resolve the links prefixed with RTC: correctly?

@davidbrochart
Copy link
Collaborator

I must say it's not very clear to me why the RTC drive could not register itself as the default JupyterLab drive in the first place, removing the need for the RTC: prefix.

@martinRenou
Copy link
Member

I must say it's not very clear to me why the RTC drive could not register itself as the default JupyterLab drive in the first place, removing the need for the RTC: prefix.

I think I'd go in the other direction, saying I don't understand why the default drive (the one providing the content from the server) shouldn't be treated as any other drives, prefixing its path properly.

@davidbrochart
Copy link
Collaborator

But still, it feels there should be a notion of a "default drive", right? So the issue would remain, could the RTC drive register itself as this default drive?

@martinRenou
Copy link
Member

martinRenou commented Feb 26, 2024

could not register itself as the default JupyterLab drive in the first place,

That's actually the thing, the "default drive" does not register itself as a drive, it's not a drive (in the sense of JupyterLab).

I believe it's the root cause of extensions assuming drives are not a thing and paths do not specify the drive. This design probably does not make the custom drive approach very welcome.

@davidbrochart
Copy link
Collaborator

So what about making the "default drive" a proper drive, which would come with a default implementation in JupyterLab, and that could be swapped by an extension?

@martinRenou
Copy link
Member

So what about making the "default drive" a proper drive, which would come with a default implementation in JupyterLab, and that could be swapped by an extension?

💯 for that

@davidbrochart
Copy link
Collaborator

I think https://github.com/davidbrochart/jupyterlab-remote-contents goes in that direction.

@loichuder
Copy link

Yes it probably doesn't support custom drives.

I'm not going to fix every extension, it might be better to open an issue there.

As far as jupyterlab-h5web is concerned, I'll be happy to add support for custom drives myself but I cannot find a documentation link nor an example for this.

Could JupyterLab (or the collaboration extension) provide a compatibility layer router to resolve the links prefixed with RTC: correctly?

That would be super helpful for extension maintainers 👍

@andreytaboola
Copy link

Any updates on the fix for this bug?
We also have issues opening notebooks using "notebook" and "NbClassic"
Thank you

@davidbrochart
Copy link
Collaborator

There is ongoing work.

@kzgrzendek
Copy link

Hello @davidbrochart, thanks for your inputs on this. Do you know if there is a landing release date for this fix? We have multiple users affected by this behavior that breaks parts of extensions or features :/

If you'd like us to test anything, I'd be more than happy to help :)

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 a pull request may close this issue.