Skip to content
This repository has been archived by the owner on Jun 27, 2023. It is now read-only.

Support Other CGI Modules #2

Open
srguglielmo opened this issue Jun 1, 2017 · 2 comments
Open

Support Other CGI Modules #2

srguglielmo opened this issue Jun 1, 2017 · 2 comments
Assignees

Comments

@srguglielmo
Copy link
Owner

srguglielmo commented Jun 1, 2017

  • mod_fastcgi, a 3rd party Apache module released under a custom open-source license. Website offline, not sure if this is supported upstream (but there seem to be forks around).
  • mod_fcgid, originally 3rd party, but moved to Apache core in 2009.
  • mod_proxy_fcgi, new in Apache 2.4, officially part of Apache core.

See mitcho/shibboleth#13

@srguglielmo
Copy link
Owner Author

Hi @jrchamp and @cjbnc,

I believe e0d52eb should fix this issue by allowing the customization of attributes. Could you take a look and let me know if it addresses the issue?

Thanks!
Steve

@jrchamp
Copy link

jrchamp commented Aug 21, 2017

The two functions referencing the private member variable should use $this->Attributes.

By customization, you still mean "edit the code to set new hardcoded values". It's in one place instead of two now, but that does not reduce the overall level of difficulty to maintain across disparate systems.

As far as addressing the issue, I would say 'no'. We use a standard configuration for the Shibboleth attributes and this Issue is about supporting multiple modules simultaneously, so by not applying the same transformations as in the original patch to the configured attributes names, it cannot work for multiple modules without also making changes to the code to match the chosen module.

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

No branches or pull requests

2 participants