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.

Sign up for GitHub

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

Jump to bottom

Use basic authentication where sending email and password is necessary #37

Closed
aron123 opened this issue Dec 5, 2020 · 2 comments · Fixed by #46
Closed

Use basic authentication where sending email and password is necessary #37

aron123 opened this issue Dec 5, 2020 · 2 comments · Fixed by #46

Comments

@aron123
Copy link
Owner

aron123 commented Dec 5, 2020

@aron123 aron123 mentioned this issue Dec 10, 2020
Closed
@aron123
Copy link
Owner Author

aron123 commented Dec 26, 2020 •
edited
Loading

If validateModels is turned off, fetch-api.js does not know the exact name of username and password fields (e.g. it can be either UsErNamE, UserName, Password, PassWoRd, password, stc.) and there can be multiple username and password fields in the request body. So, when the pre-validation is turned off, fetch-api.js cannot exactly choose the right fields of the request object for the basic authentication.

Using a different approach in Barion options is necessary (e.g. Setting Secure: false instead of ValidateModels: false).

@aron123
Copy link
Owner Author

aron123 commented Dec 26, 2020 •
edited
Loading

  • ValidateModels -> Secure
  • Documentation of initial options
  • Documentation of sending credentials

@aron123 aron123 mentioned this issue Dec 27, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use Basic Authentication for sending username and password aron123/node-barion
1 participant
@aron123