Skip to content
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

feat: add cedar-wasm and parsing/validation features #156

Merged
merged 5 commits into from
Jul 1, 2024
Merged

feat: add cedar-wasm and parsing/validation features #156

merged 5 commits into from
Jul 1, 2024

Conversation

Swolebrain
Copy link
Contributor

  • Adds cedar-wasm
  • Fixes some vulnerability that npm found
  • Gets rid of Statement class in favor of more robust functionality which includes parsing and validating policies/schemas
  • Added feature to read a whole directory and load all .cedar files into policy store

reste85
reste85 requested changes Jun 26, 2024
View reviewed changes
Copy link
Contributor

@reste85 reste85 left a comment

Choose a reason for hiding this comment

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

minor change

src/policy-store.ts Outdated Show resolved Hide resolved
auto-merge was automatically disabled June 26, 2024 12:18

Head branch was pushed to by a user without write access

reste85
reste85 approved these changes Jun 27, 2024
View reviewed changes
Copy link
Contributor

@reste85 reste85 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks a lot for the amazing contribution!

@reste85 reste85 added the enhancement New feature or request label Jun 27, 2024
@svenjaraether svenjaraether added this pull request to the merge queue Jul 1, 2024
Merged via the queue into cdklabs:main with commit 8668dac Jul 1, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@reste85 reste85 reste85 approved these changes

Assignees

@Swolebrain Swolebrain

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Swolebrain @reste85 @svenjaraether