Technology Admin

The maintainer's philosophy seems to be "simplest possible" so additional settings probably won't get accepted. The change I propose should be at least as secure and address your case as...

I want/need the same. It'd be nice to include a stock method that does the update. I ended up with: ``` def update_user(user_identity): user_email = user_identity[settings.SAML2_AUTH.get('ATTRIBUTES_MAP', {}).get('email', 'Email')][0] user_name =...

@kherrett SAML2_AUTH = { ... 'TRIGGER': { 'BEFORE_LOGIN': 'app_name.saml.update_user', }, I assume this method works in the base application, but I'm now working on a [fork/branch](https://github.com/ambsw/django-saml2-auth/tree/plugin-system) that was refactored to...

I have a really rough refactor on our [plugin-system](https://github.com/ambsw/django-saml2-auth/tree/plugin-system) branch and an example of how my [inline metadata PR](https://github.com/fangli/django-saml2-auth/pull/67) would be (re)implemented as a plugin in the [django-saml2-auth-metadata-inline](https://github.com/ambsw/django-saml2-auth-metadata-inline) repo. In...

There are open PRs in this library from @lllama @aligx @cochiseruhulessin @favorable-mutation @velapartners @apexJCL @anthonyeden @liamkinney @chriskj @luzik @alexazevedo @avallbona @jianyuan @pappacena @kronion @alxbridge @kevPo @kronion @HappyTepid @jheld. Would any...

When a user goes to the `signin` endpoint, they should be redirected to a SAML IdP with a payload. This option indicates that the payload should not be signed. Per...

merge https://github.com/fangli/django-saml2-auth/pull/68

I *strongly* suggest moving the methods back to their original locations. Relocating methods like this makes the PR incompatible with all others.

I don't have strong feelings for or against reorganizing, but you can't combine relocated code with other changes in a community repo like this. 1. Combining different issues in one...

Yeah. I had to fix the `reverse` code *because* I'm not using admin so I'm with you that it's not a great default (today, let alone with your proposed defaults)....