Basic Auth for multiple users.

[hkiiita]

This PR adresses Basic Auth #330 for multiple users. Please review.

[hkiiita]

@dfarr Please have a look.

[dfarr]

Sorry for the delay in reviewing @hkiiita , this looks great! I have one request, rather than have a separate file for username/password information could we change this to use the default configuration file? The file could look like this:

auth:
  - username: user1
    password: pass1
  - username: user2
    password: pass2

[hkiiita]

Thanks for having a look @dfarr ; please review the new changes. I have removed the separate creds file as asked and am now reading credentials from the main config file itself.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.10%. Comparing base (32bd3b7) to head (5935351).
Report is 5 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #352      +/-   ##
==========================================
- Coverage   58.12%   58.10%   -0.03%     
==========================================
  Files         113      113              
  Lines        9943     9938       -5     
==========================================
- Hits         5779     5774       -5     
  Misses       3796     3796              
  Partials      368      368              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dfarr]

Thank you @hkiiita! Now that we are using viper for the configuration we shouldn't need to do manual parsing of the configuration, as long as we specify the type I believe viper will parse automatically for us.

[dfarr]

🎉

[hkiiita]

🎉

Thank you so much @dfarr for the journey. I definitely learned a couple of things during this. Thanks again !