Addition of basic Auth functionality as mentioned in issue Basic Auth #330

[hkiiita]

Addition of basic Auth functionality as mentioned in issue Basic Auth #330

[hkiiita]

Incorporates multiple users.

[dfarr]

Thank you so much for the PR @hkiiita 🎉

We can't believe it, but we actually got two PRs for the same feature. We are going to merge #332, but would you be up for incorporating multiple users (which is lacking in #332) once that one is merged?

[hkiiita]

Thank you so much for the PR @hkiiita 🎉

We can't believe it, but we actually got two PRs for the same feature. We are going to merge #332, but would you be up for incorporating multiple users (which is lacking in #332) once that one is merged?

Hi @dfarr thank you so much for your kind reply on this. Yes , i will be more than happy to continue to work on this after 332 gets merged. I have already incorporated multiple users authentication into my PR #333 , so will wait on your side for a review and further steps. Thanks again !

[dfarr]

Hey @hkiiita, PR #332 has been merged, if you're still up for it, we could extend basic auth to support multiple users - I believe we would want to extend these arguments to take an array of username/password combos (and propagate that array down accordingly)

[hkiiita]

Hi @dfarr , thanks for the reply . Yes I definitely would want to work on this . Just wanted to share that, if you are ok with it , I will reach out to you on 18th June.

[hkiiita]

#352