Skip to content
This repository has been archived by the owner on Jan 6, 2018. It is now read-only.

[3.0] Gh 0035 Oozie authentication module #330

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

angeloh
Copy link

@angeloh angeloh commented Jan 7, 2011

GH-35 Oozie authentication module

@angeloh
Copy link
Author

angeloh commented Jan 26, 2011

Andreas,

  1. The token is not required to be there. It can be null.
  2. It will pick only one provider and do the authenticate(). If it fails, return exception. A server could contain many providers and client's connection might hang if continue to try rest of providers.
  3. Yes, providers are created statically. The configuration can be defined in oozie-site.xml and read it within the provider. For example, you can put one configuration 'oozie.auth.a.xxx' in oozie-site.xml, when this provider is instantiated by AuthenticationProviderFactory, the value can be read from the argument 'Configuration'.

@angeloh
Copy link
Author

angeloh commented Jan 27, 2011

Hi Tucu,

Comments for your questions:

  1. OozieAuthFilter is getting the value by calling HttpServletResponseWrapper.getUserPrincipal(). I am not sure if I answer your question correctly?
  2. wiki is created here. https://github.com/yahoo/oozie/wiki/Oozie-Authentication-Specification
    Also, it is in Oozie docs too.

3&4 are addressed in last two commits. Client code too.

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

Successfully merging this pull request may close these issues.

1 participant