-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat: support LOGIN auth mechanism #15
Conversation
I have added a few comments, mostly minor things. @paskal pls take a look too, thx |
@paskal hi! I have resolved all umputun comments. Please review it |
I'll review it this evening. In the meantime, could you please run |
1f2d5f6
to
a345f4d
Compare
I have resolved golangci-lint errors. thx |
@paskal hello! |
@umputun let's merge? ) |
sure. Just two more things before we merge:
|
- use enum - simplify options interface
Currently, only two authentication methods are supported. So for now, switch can be dropped.
To be consistent with other options
@umputun now this feature branch is rebased and ready to be squashed and merged |
@umputun I forgot to write right away, I added an info about new auth option to README.md. Looks like you can merge the PR. |
Closes #14