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
Docs: Add limitations for Basic Authentication #26817
Conversation
Hi @hmanwani-rh Nice work, I've added a few minor suggestions which I thought could clarify a few details better, have a look please |
@sberyozkin Thanks for reviewing the update and sharing the comments. I've incorporated the changes as suggested. |
Good Job @hmanwani-rh . Will add my review now. Will add just few lines to make it complete. |
LGTM |
9a91e58
to
a189ee7
Compare
Thanks @hmanwani-rh I've just squashed a few last commits with |
I think they will need to have it all in a single commit only. @hmanwani-rh |
@MichalMaler I kept the last 2 ones as I did not want a reference to your contribution be lost :-), I will squash now to the single commit as looks like you are OK with it :-) |
a189ee7
to
294b5a3
Compare
@hmanwani-rh @MichalMaler All is ready now, thanks for your contribution :-) |
Oh well, hope it will be written forever in our hearts :D :D Thank you for the attitude thought! Well done, We! |
See JIRA: https://issues.redhat.com/browse/QDOCS-29
Added limitations for Basic Authentication in "security-built-in-authentication.adoc" file.
Doc preview: http://file.pnq.redhat.com/~hmanwani/QDOCS-29-main