sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Colin Ma <junjie...@intel.com>
Subject Re: Review Request 42785: SQOOP-2807: Sqoop2: Add admin user list to configuration file
Date Sun, 14 Feb 2016 08:45:25 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42785/
-----------------------------------------------------------

(Updated Feb. 14, 2016, 8:45 a.m.)


Review request for Sqoop and Colin Ma.


Bugs: SQOOP-2807
    https://issues.apache.org/jira/browse/SQOOP-2807


Repository: sqoop-sqoop2


Description
-------

Add admin user list to configuration file, and Sqoop can do the admin authorization according
to the value.
For example:
 org.apache.sqoop.security.admins=admin1,admin2


Diffs (updated)
-----

  core/src/main/java/org/apache/sqoop/security/AuthorizationManager.java 4afdf02 
  core/src/main/java/org/apache/sqoop/security/SecurityConstants.java 6f32e04 
  dist/src/main/conf/sqoop.properties 620146d 
  server/src/main/java/org/apache/sqoop/filter/SqoopAdminCheckFilter.java PRE-CREATION 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyServer.java 2c4cb7a 

Diff: https://reviews.apache.org/r/42785/diff/


Testing
-------


Thanks,

Colin Ma


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message