Skip to content
This repository has been archived by the owner on Aug 11, 2020. It is now read-only.

Commit

Permalink
Trying to get security working... still failing
Browse files Browse the repository at this point in the history
  • Loading branch information
jdillon committed Apr 16, 2013
1 parent b0f13c7 commit 9fc5aee
Showing 1 changed file with 5 additions and 6 deletions.
Expand Up @@ -24,7 +24,7 @@
/** /**
* <a href="http://metrics.codahale.com">Yammer Metrics</a> guice configuration. * <a href="http://metrics.codahale.com">Yammer Metrics</a> guice configuration.
* *
* Installs metrics endpoints: * Installs servlet endpoints:
* *
* <ul> * <ul>
* <li>/internal/ping</li> * <li>/internal/ping</li>
Expand All @@ -33,7 +33,7 @@
* <li>/internal/healthcheck</li> * <li>/internal/healthcheck</li>
* </ul> * </ul>
* *
* Protected by {@code metrics-endpoints} permission. * Protected by {@code nexus:metrics-endpoints} permission.
* *
* @since 2.5 * @since 2.5
*/ */
Expand Down Expand Up @@ -94,14 +94,13 @@ protected void configureServlets() {
} }
}); });


// require permission to use endpoints
install(new FilterChainModule() install(new FilterChainModule()
{ {
@Override @Override
protected void configure() { protected void configure() {
addFilterChain(MOUNT_POINT + "/**", "noSessionCreation,authcBasic"); // FIXME: This is working slighlty better now, but still not working properly :-(

addFilterChain(MOUNT_POINT + "/**", "noSessionCreation,authcBasic,perms[nexus:metrics-endpoints]");
// FIXME: This does not work, unsure why...
//addFilterChain(MOUNT_POINT + "/**", "noSessionCreation,authcBasic,perms[metrics-endpoints]");
} }
}); });


Expand Down

0 comments on commit 9fc5aee

Please sign in to comment.