Github user bhairavi25 commented on a diff in the pull request:
https://github.com/apache/incubator-brooklyn/pull/74#discussion_r15271277
--- Diff:
usage/rest-server/src/main/java/brooklyn/rest/resources/ServerResource.java ---
@@ -93,4 +95,14 @@ public HighAvailabilitySummary getHighAvailability() {
ManagementPlaneSyncRecord memento =
mgmt().getHighAvailabilityManager().getManagementPlaneSyncState();
return
HighAvailabilityTransformer.highAvailabilitySummary(mgmt().getManagementNodeId(),
memento);
}
+
+ @Override
+ public String getUser() {
+ EntitlementContext entitlementContext =
Entitlements.getEntitlementContext();
+ if (entitlementContext!=null && entitlementContext.user()!=null){
+ return entitlementContext.user();
+ } else {
+ return null; //User can be null if no authentication was requested
--- End diff --
Thankyou for reviewing @sjcorbett !
Just curious, in your eclipse development do you use any formatting
config/plugin which does formatting - spacing and indentation with
say(Ctrl+Shift+F). Some of the formatting issues looks good in eclipse though
not in github. Added four spaces from the beginning and for any code after
every bracing.
New review comment changes:
https://github.com/bhairavi25/incubator-brooklyn/commit/fc261b70865d045204ad81c80055224b0926174e
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
