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

pass headers into authenticate #49

Closed
pyramation opened this issue Dec 14, 2018 · 1 comment · Fixed by #52
Closed

pass headers into authenticate #49

pyramation opened this issue Dec 14, 2018 · 1 comment · Fixed by #52
Assignees

Comments

@pyramation
Copy link

pyramation commented Dec 14, 2018

Would be great if the authenticate method had access to request headers. Looks like it wouldn't be terribly hard since the req object is already right there:

const promise = this.authenticate(type, repoName, basicAuth.bind(null, req, res), (error) => {

@gabrielcsapo
Copy link
Owner

Thanks for looking into the PR @pyramation, it is released in 0.6.0 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants