Skip to content
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

Add a function that generates an SP initiated SSO request #19

Merged
merged 1 commit into from Sep 22, 2022

Conversation

fumieval
Copy link
Contributor

This PR adds a basic support for SP-initiated flow. I confirmed that it works with Google Workspace

Copy link
Owner

@mbg mbg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like most of new the Haskell file is formatted using Ormolu (or similar), although there is some inconsistent indentation. It would be good if the formatting could be more in line with the other files, as per the contribution guidelines for style.

wai-saml2.cabal Outdated Show resolved Hide resolved
src/Network/Wai/SAML2/Request.hs Outdated Show resolved Hide resolved
src/Network/Wai/SAML2/Request.hs Outdated Show resolved Hide resolved
src/Network/Wai/SAML2/Request.hs Outdated Show resolved Hide resolved
@fumieval fumieval force-pushed the sp-initiated branch 2 times, most recently from b07ddac to c09c639 Compare September 21, 2022 04:16
@mbg
Copy link
Owner

mbg commented Sep 22, 2022

@fumieval Thanks for your work on this and the changes you have made in response to the review. It seems that the formatting and documentation are still outstanding. Is that something you are still planning to do? If not, I can deal with the documentation and re-format the code.

@fumieval fumieval force-pushed the sp-initiated branch 2 times, most recently from 2fcfc64 to 6379cb9 Compare September 22, 2022 08:57
@fumieval
Copy link
Contributor Author

I'm afraid I'm not really sure what the issues with documentation and formatting are. Could you reformat the code?

@mbg mbg merged commit 45c45df into mbg:master Sep 22, 2022
@mbg
Copy link
Owner

mbg commented Sep 22, 2022

No problem and thanks for this contribution! :) I have merged it and pushed my changes on top. You may wish to have a look over those to make sure all is good with you as well.

@fumieval
Copy link
Contributor Author

Thank you for making up the documentation!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants