Skip to content

Bullboard auth #11

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

Merged
merged 5 commits into from
Sep 4, 2021
Merged

Bullboard auth #11

merged 5 commits into from
Sep 4, 2021

Conversation

VidithM
Copy link
Contributor

@VidithM VidithM commented Aug 28, 2021

For issue #9. Now, whenever the background is running and we go to /bull-board, if we are not authenticated we are first redirected to a login page. The login credentials can be configured in .env.

@SysSn13
Copy link
Owner

SysSn13 commented Aug 28, 2021

Please remove unnecessary changes. I think you recompiled the node addon.

@VidithM
Copy link
Contributor Author

VidithM commented Aug 29, 2021

How about now?

@SysSn13
Copy link
Owner

SysSn13 commented Aug 29, 2021

You have deleted too much stuff, unstage those changes instead of deleting files.

@SysSn13
Copy link
Owner

SysSn13 commented Aug 29, 2021

This is not how it works. You should reset your commits and then stage only desired changes.

@VidithM
Copy link
Contributor Author

VidithM commented Aug 29, 2021

Sorry, I am working on it. I'll comment when I get the right version

@VidithM
Copy link
Contributor Author

VidithM commented Aug 29, 2021

Ok, I think it should be good now? Let me know.

@SysSn13 SysSn13 linked an issue Sep 2, 2021 that may be closed by this pull request
@VidithM
Copy link
Contributor Author

VidithM commented Sep 2, 2021

I also added a 404 page for invalid paths

@SysSn13 SysSn13 merged commit b0bd7ef into SysSn13:main Sep 4, 2021
SysSn13 added a commit that referenced this pull request Jan 30, 2024
* Added bull-board auth using passport.js

* Minor changes

* update .gitignore

Co-authored-by: SysSn13 <sudesh18@iitg.ac.in>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add authentication for bull-board
2 participants