GKaszewski/loco-keycloak-auth
1
0
Fork 0
Code Issues 5 Pull Requests Actions Packages Projects Releases Wiki Activity

Fix doc examples to use ? instead of .expect() #5

New Issue
Open
opened 2026-07-25 11:16:30 +00:00 by GKaszewski · 0 comments
GKaszewski commented 2026-07-25 11:16:30 +00:00
Owner
Copy Link

Severity: Low

The usage example in Keycloak::from_context doc comment uses .expect("Failed to create Keycloak layer") which will panic in production if Keycloak is misconfigured.

Doc examples should demonstrate ? or proper error handling to set the right example for consumers.

**Severity: Low** The usage example in `Keycloak::from_context` doc comment uses `.expect("Failed to create Keycloak layer")` which will panic in production if Keycloak is misconfigured. Doc examples should demonstrate `?` or proper error handling to set the right example for consumers.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GKaszewski
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: GKaszewski/loco-keycloak-auth#5
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?