-
Notifications
You must be signed in to change notification settings - Fork 158
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
Fixing issue with package notebook in jupyterhub image. #785
Conversation
german-grandas
commented
Aug 27, 2024
- Refactoring of code so it can build and install directly the authentication package instead of getting it from github.
- Installing notebook as a separate dependency
- Letting pip install the jupyterhub version according to the notebook package.
- Refactoring of code so it can build and install directly the authentication package instead of getting it from github. - Installing notebook as a separate dependency - Letting pip install the jupyterhub version according to the notebook package.
/gcbrun |
/gcbrun |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi german-grandas, thanks for contributing to the repo! Just have a few questions and comments.
USER root | ||
RUN chmod -R u+w ./authenticator | ||
|
||
RUN pip3 install notebook |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the reason for installing the notebook for the hub image?
license='Apache 2.0', | ||
install_requires=[ | ||
"jupyterhub>=4.1.0", | ||
"jupyterhub", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe version 4.1.0 contains a fix for a known vulnerability and thats why we have `>=4.1.0"
Closing PR, the bug is fixed! |