Skip to content

feat(rust/rbac-registration): RBAC refactoring feature branch #630

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

feat(rust/rbac-registration): RBAC refactoring feature branch #630

wants to merge 6 commits into from
+498 −161

Conversation

@Mr-Leshiy
Copy link
Contributor

@Mr-Leshiy Mr-Leshiy commented Nov 7, 2025 •
edited
Loading

Description

Related Pull Requests

Please confirm the following checks

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream module

@apskhem @Mr-Leshiy
feat(rust/rbac-registration): Add provider trait for `RegistrationCha…
ac4093c 
…in` (#586)

* initial

* feat: provider

* feat: update chain

* feat: start_new_chain

* feat: merge validation result struct into cip509

* feat: exports

* feat: return new chain

* chore: return success object

* fix: older version

* feat: cat id in payload

* fix: new version

* chore: lintfix

* chore: remove persistent arguments

* tmp

* chore: complete moving to central module

* feat: ref fn

* chore: merge methods

* chore: minor

* feat: export modified chains

* chore: minor comment

* chore: rbac update logic

* chore: isolation

* chore: validation and lintfix

* docs: remove error doc

* chore: minor refactor

* fix: comments

* chore: validation function return

* Update rust/rbac-registration/src/providers.rs

Co-authored-by: Alex Pozhylenkov <leshiy12345678@gmail.com>

---------

Co-authored-by: Alex Pozhylenkov <leshiy12345678@gmail.com>
@Mr-Leshiy Mr-Leshiy added the draft Draft label Nov 7, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

📚 Docs Preview

The docs for this PR can be previewed at the following URL:

https://docs.dev.projectcatalyst.io/libs/feat/rbac

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025 •
edited
Loading

✅ Test Report | ${\color{lightgreen}Pass: 566/566}$ | ${\color{red}Fail: 0/566}$ |

@Mr-Leshiy
feat(rust/rbac-registration): RBAC stolen StakeAddress handling. (#631
5c0b665 
)

* wip

* wip

* wip

* wip

* revert

* wip

* add stolen_uris field

* wip

* wip

* wip

* fix

* wip

* wip

* fix spelling

* fix

* wip

* wip

* wip

* wip

* cleanup

* fix test

* fix clippy

* wip

* wip

* fix

* wip

* fix clippy

* fix

* wip

* wip
@Mr-Leshiy
Merge branch 'main' into feat/rbac
7781517
@Mr-Leshiy Mr-Leshiy added review me PR is ready for review squad: gatekeepers Catalyst App Backend, System Development & Integration Team do not merge yet PR is not ready to be merged yet and removed draft Draft labels Nov 19, 2025
@Mr-Leshiy Mr-Leshiy moved this from New to 👀 In review in Catalyst Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stanislav-tkach stanislav-tkach Awaiting requested review from stanislav-tkach

@bkioshn bkioshn Awaiting requested review from bkioshn

@rafal-ch rafal-ch Awaiting requested review from rafal-ch

@apskhem apskhem Awaiting requested review from apskhem

At least 1 approving review is required to merge this pull request.

Assignees

@Mr-Leshiy Mr-Leshiy

Labels

do not merge yet PR is not ready to be merged yet review me PR is ready for review squad: gatekeepers Catalyst App Backend, System Development & Integration Team

Projects

Catalyst
Status: 👀 In review

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Mr-Leshiy @apskhem