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

"Make a copy" no longer works on read-only notebooks #2541

Closed
dietmarw opened this issue Jun 1, 2017 · 7 comments · Fixed by #2854
Closed

"Make a copy" no longer works on read-only notebooks #2541

dietmarw opened this issue Jun 1, 2017 · 7 comments · Fixed by #2854

Comments

@dietmarw
Copy link

dietmarw commented Jun 1, 2017

When editing a read-only jupyter notebook (made read-only via file system) I noticed that I can no longer use the "Make a copy" feature. All that happens is that a new empty tab is opened.

I played a bit around to find out when this happens exactly.

  1. One can get lucky if one is quick enough to hit the button before Jupyter is finished rendering the notebook.
  2. I could only reproduce this issue when the Jupyter Notebook is served via a port reroute (using iptables from port 443 or 80 to 8888) via web. On a local installation this does not happen.

As a note this did not use to be an issue like half a year ago were I ran the same set of Jupyter Notebooks with the exact same set up. I can be sure of that since I used the same docker setup.

I can supply a running live server where this can tested out if needed.

@takluyver
Copy link
Member

I'm not sure why the port rerouting would affect it, but if the JS considers that the notebook is dirty (changed since last save), it tries to save it before copying. I think the issue is likely that when the kernel starts, it updates the kernel metadata and marks the notebook as dirty. Maybe we should just not count that change as dirty-ing the notebook.

@dietmarw
Copy link
Author

dietmarw commented Jun 2, 2017

Yes I'm not too sure about the influence of port reroute myself. I tested it again disabling the reroute to port 443 and accessing it directly via the default port 8888 gives the same (non-working) result.

@dietmarw
Copy link
Author

dietmarw commented Aug 18, 2017

Just tested it again and the issue is still present.

@minrk
Copy link
Member

minrk commented Aug 20, 2017

Yeah, we have a few 'save first' actions (trust was just addressed recently in #2718) that should probably be "save first unless readonly".

@Madhu94
Copy link
Contributor

Madhu94 commented Aug 20, 2017

I'll check this.

@dietmarw
Copy link
Author

Any update on this issue?

@dietmarw
Copy link
Author

Excellent, thanks!

@minrk minrk added this to the Reference milestone Sep 13, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants