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

contrib directory + 1 example from community so far #2

Closed
wants to merge 1 commit into from
Closed

contrib directory + 1 example from community so far #2

wants to merge 1 commit into from

Conversation

jblaine
Copy link

@jblaine jblaine commented Feb 15, 2012

No description provided.

@jblaine
Copy link
Author

jblaine commented Mar 1, 2012

Please do consider pulling this. It's unobtrusive to the codebase, won't stop anything from functioning for you, is from a working setup, and as shown by the questions on rt-users, this extension is not well understood by folks. I have another to submit from a community member, but I'm not going to bother submitting it if this sort of thing is not welcome.

It's very easy to say "there should just be better docs!", but something is better than nothing. I don't see "a thorough guide to externalauth" coming out of Best Practical any time soon, so...

@tsibley
Copy link
Contributor

tsibley commented May 15, 2013

There are various definitions of what "working" means for a config. An AD admin who reasonably expects the standard disabled user flag in AD to be respected by RT would be quite surprised by the contributed config in this PR. We'd prefer not to distribute (any more) configs which contain know trip points or inaccuracies. Distributing them, even in a contrib/ directory, implicitly stamps our approval/review on it.

What we would like is a good library of general example configs for various scenarios, such as AD. If you'd like to fixup the points I mentioned a year ago and approach configs from this angle, that'd be great.

Please also take a look at the docs-improvements branch where we're reworking the ExternalAuth docs.

@jblaine jblaine closed this May 16, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants