Skip to content

Commit

Permalink
fix(Logging) : Log less page details when making an unauthorized Grap…
Browse files Browse the repository at this point in the history
…hQL request #29651 (#29652)

### Proposed Changes
* Change log to only display `htmlPageAsset.getURI()`

### Screenshots
### Before:

![image](https://github.com/user-attachments/assets/bb3ea6cd-cadf-45fb-9234-4706563928b7)
### After:

![image](https://github.com/user-attachments/assets/fe5b293d-0239-4b3a-af71-77b08cc7589a)

This PR fixes: #29651
  • Loading branch information
valentinogiardino authored Aug 20, 2024
1 parent 600b4f3 commit bffd903
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -377,9 +377,9 @@ private void checkPagePermission(final PageContext context, final IHTMLPage html
context.getPageMode().respectAnonPerms);

if (!doesUserHavePermission) {
final String message = String.format("User: %s does not have permissions %s for object %s",
final String message = String.format("User: %s does not have permissions %s for page %s",
context.getUser(),
PermissionLevel.READ, htmlPageAsset);
PermissionLevel.READ, htmlPageAsset.getURI());
throw new DotSecurityException(message);
}
}
Expand Down

0 comments on commit bffd903

Please sign in to comment.