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

custom_display_url changes only one URL #5400

Closed
liamhuber opened this issue Apr 24, 2020 · 4 comments
Closed

custom_display_url changes only one URL #5400

liamhuber opened this issue Apr 24, 2020 · 4 comments

Comments

@liamhuber
Copy link

First, sorry if I'm posting this in the wrong sub-repo.

My issue is that after setting c.NotebookApp.custom_display_url = 'http://mycustomurl/', it changes only the first url in the user output, e.g.

    Or copy and paste one of these URLs:
        http://mycustomurl/?token=[a token string]
     or http://127.0.0.1:8888/?token=[a token string]

In my particular case I'm just changing the port to 8009, and the result is localhost:8009 and 127.0.0.1:8888 both appearing.

Would it be possible to show only one link whenever custom_display_url is set to something non-empty?

I tracked down the issue where this url display was decided on, but couldn't find any more recent related issues.

@kevin-bates
Copy link
Member

Hi @liamhuber - I apologize for the delayed response - this is the correct place to raise this issue.

Your issue makes sense to me. Did you want to submit a pull request for this?

LaurMerl pushed a commit to LaurMerl/notebook that referenced this issue Jun 20, 2020
LaurMerl pushed a commit to LaurMerl/notebook that referenced this issue Jun 20, 2020
LaurMerl pushed a commit to LaurMerl/notebook that referenced this issue Jun 20, 2020
kevin-bates added a commit that referenced this issue Jun 24, 2020
FIX: removed double link when custom_display_url (#5400)
@kevin-bates
Copy link
Member

Closed via #5544

@liamhuber
Copy link
Author

@kevin-bates @LaurMerl,

Thanks for the fix and merge! Thanks for your patience on the gratitude too -- I'm on parental leave atm so my timelines for work stuff are super stretched out.

@kevin-bates
Copy link
Member

No worries @liamhuber - thanks for raising the issue. I'm sure there will be other opportunities to contribute to Jupyter.

Congratulations on your new addition! 🎉

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants