mirror of
https://github.com/axllent/mailpit.git
synced 2026-04-26 00:35:51 +03:00
[GH-ISSUE #1] Basic Auth #2
Labels
No labels
awaiting feedback
bug
docker
documentation
enhancement
github_actions
invalid
pull-request
question
stale
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/mailpit#2
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @mbuk on GitHub (Aug 3, 2022).
Original GitHub issue: https://github.com/axllent/mailpit/issues/1
Hi @axllent thanks for releasing this.
Can I +1 for basic auth support?
@axllent commented on GitHub (Aug 3, 2022):
I had actually already started work on this feature yesterday, so you can likely expect it in the next couple few days 👍
@mbuk commented on GitHub (Aug 3, 2022):
Amazing 🙇
@axllent commented on GitHub (Aug 3, 2022):
I'll leave this open until it is actually implemented and tested :)
@axllent commented on GitHub (Aug 4, 2022):
@mbuk I've just release 0.0.5 which contains Basic Auth support. Please test and report any issues, thanks.
@mbuk commented on GitHub (Aug 4, 2022):
I'll check it out today!
Thanks
@mbuk commented on GitHub (Aug 4, 2022):
@axllent noticed that the auth file spec has -a, ---auth-file string when looking at the help
but the documentation says --auth-file
Triple hyphen in exec but double in docs.
Other that minor point all working perfectly!
Tested changing the hash type
Tested multiple users
Tested file missing
Tested permissions restricted to file.
All behaves as expected!
Thank you!
@axllent commented on GitHub (Aug 4, 2022):
Thanks for the feedback & testing! You are absolutely right, that's a typo in the three
---- I will fix that which will be included in the next release tomorrow.@sandervanzutphen commented on GitHub (Aug 4, 2022):
Are the different basicAuth users supposed to result in split inboxes? That does not seem to be te case atm.
This becouse the -h parameters describes the -m parameter as "Max number of messages per mailbox (default 500)", emphatis on the 'per mailbox' part.
@axllent commented on GitHub (Aug 4, 2022):
No it does not, it is all just access to one mabox. Initially I considered that, but to get something like that working would require email filtering, rules, permissions, and a bunch more. It just became too complicated for a tool intended only for developer testing.
@axllent commented on GitHub (Aug 6, 2022):
@mbuk Just to keep you in the loop as I know you're using this feature, in the latest (0.1.0) release
-aand--auth-filehave been deprecated and replaces with just--ui-auth-file(no short flag). I did this because Mailpit now also has the option for SMTP authentication which is called--smtp-auth-file- so it made the most sense to rename that original flag.@mbuk commented on GitHub (Aug 6, 2022):
@axllent Cheers 👍