-
Notifications
You must be signed in to change notification settings - Fork 156
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
Update ubuntu and sasl_xoauth2 version and add SMPTD_SASL_USERS option #116
Conversation
Hey @vcazsdk your commits look nice -- I'd like to merge them to the main tree, if you've finished with them? |
Hi @bokysan. Yes i have finished it, tested it and put it in our production. That's work well for me. You can merge to the main tree. Thank you |
I have a failure when bulding SASL XOAUTH. Have you run into this issue? https://github.com/bokysan/docker-postfix/runs/8201677523?check_suite_focus=true |
Yes i got this error with alpine build. At the time, i thought it was my environment that was incorrect. I didn't try to correct this error because I didn't have enough time and we don't use the alpine image. I had build only the amd64 ubuntu image |
I have pushed new commit that fix alpine build error |
Definitely, the alpine image is buggy with sasl-xoauth2. Authentication failed with this postfix error :
|
Hm, that might be the reason I never managed to get integration tests in #43 working. I'll merge this to master and test it out once the compile passes. Most likely will have to give up on sasl-xoauth2 in alpine until this issue is fixed upstream. Thanks so much for your work! |
No description provided.