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

Removing params restriciton for authentication parameters #13

Merged
merged 1 commit into from
May 5, 2017
Merged

Removing params restriciton for authentication parameters #13

merged 1 commit into from
May 5, 2017

Conversation

mishuagopian
Copy link
Contributor

@mishuagopian mishuagopian commented May 5, 2017 •
edited
Loading

Summary

  • Removed authentication params restriction as users may authenticate entities with whatever they want.

This solves #11.

@mishuagopian mishuagopian requested review from hdf1996 and alebian May 5, 2017 12:28
@mishuagopian mishuagopian mentioned this pull request May 5, 2017
Closed
@Lucheti29
Copy link

👏👏👏👏👏

@mishuagopian mishuagopian merged commit e5b9c45 into master May 5, 2017
@mishuagopian mishuagopian deleted the params-restriction branch May 5, 2017 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alebian alebian alebian approved these changes

@hdf1996 hdf1996 Awaiting requested review from hdf1996

Assignees

@hdf1996 hdf1996

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mishuagopian @Lucheti29 @alebian @hdf1996