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.

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

PXD-2702: Resource rules yacc #40

Merged
merged 5 commits into from
Mar 22, 2019
Merged

PXD-2702: Resource rules yacc #40

merged 5 commits into from
Mar 22, 2019

Conversation

fantix
Copy link
Contributor

@fantix fantix commented Mar 18, 2019

New Features

  • Added resource rules yacc lexer and parser
  • Implemented auth/request and auth/proxy
  • Added lazy prepared statement cache

@fantix fantix requested a review from rudyardrichter March 18, 2019 22:45
@fantix fantix force-pushed the feat/resource-rules branch 3 times, most recently from 3e67fb0 to cc55ec5 Compare March 21, 2019 21:38
fantix added 4 commits March 21, 2019 16:45
* Also completed auth/proxy
* Also used prepared statement for performance
* Rewrote the lexter to support slashes and quotes
* Faster AND/OR check
@fantix fantix force-pushed the feat/resource-rules branch from cc55ec5 to 18df587 Compare March 21, 2019 21:46
* Added generic lazy prepared statement cache
* Used Request/Response structs in authorize()
@fantix fantix force-pushed the feat/resource-rules branch from 268c154 to 009ef77 Compare March 21, 2019 23:29
@fantix fantix merged commit 256d491 into feat/db Mar 22, 2019
@fantix fantix deleted the feat/resource-rules branch March 22, 2019 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants