Skip to content

[WIP] Add RBAC-based authentication and authorization#25

Closed
Copilot wants to merge 1 commit into080from
copilot/sub-pr-20
Closed

[WIP] Add RBAC-based authentication and authorization#25
Copilot wants to merge 1 commit into080from
copilot/sub-pr-20

Conversation

Copy link
Contributor

Copilot AI commented Mar 2, 2026 •
edited
Loading

  • rbacMiddleware.ts: Replace console.warn/console.error with LoggerService
  • expertMode.ts: Fix re-evaluation logic using Object.defineProperty for dynamic auth check
  • BoltPlugin.ts: Remove duplicate JSDoc comment block for getGroups()
  • ssh/config.ts: Remove unused SSHErrorType import
  • migrations/006: Add 'plan' to executions_new type CHECK constraint
  • DatabaseService.ts: Remove ineffective PRAGMA page_size
  • AnsibleService.ts: Use NodeGroup[] return type instead of inline anonymous type
  • .env.example: Fix typo "Allowd" → "Allowed"
  • Dockerfile: Fix typo "Awesaome" → "Awesome"
  • README.md: Fix typo "RBC" → "RBAC"
  • PuppetDBService.ts: Remove redundant null check for this.client
  • securityMiddleware.ts: Log warning when max depth exceeded instead of silently returning unsanitized obj

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI mentioned this pull request Mar 2, 2026
Merged
@alvagante alvagante marked this pull request as ready for review March 2, 2026 12:15
Copilot AI review requested due to automatic review settings March 2, 2026 12:15
Copilot AI reviewed Mar 2, 2026
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@alvagante alvagante closed this Mar 2, 2026
@alvagante alvagante deleted the copilot/sub-pr-20 branch March 2, 2026 12:16
Copilot stopped work on behalf of alvagante due to an error March 2, 2026 12:16
The session was cancelled by the user.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

@alvagante alvagante

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alvagante