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/db: add database persistence #42

Merged
merged 125 commits into from
May 14, 2019
Merged

feat/db: add database persistence #42

merged 125 commits into from
May 14, 2019

Conversation

rudyardrichter
Copy link
Contributor

@rudyardrichter rudyardrichter commented Mar 22, 2019 •
edited
Loading

Overhaul arborist server to persist information to a postgres database.

New Features

  • db migration & utilities
    • Add db migration automation and use this for the travis setup so travis is actually running tests on postgres backend
    • Add documentation on migration commands & utility scripts
  • Add CRUD endpoints for users & groups as well as granting policies
  • Add tests for REST API & database operations
    • Add documentation & examples on tests
  • Add resource rules lexer & parser for handling simple logic in auth requests

Deployment changes

  • Database deployment
    • See migrations folder for database schema and setup utilities

Rudyard Richter and others added 30 commits February 25, 2019 10:10
feat(db): work on v0 migrations
d7a49fc
feat(db): work on db schema
530567a
feat(db): add endpoints for models
bab2416
feat(db): add token decoder
d339260
feat(db): add grp_policy table
7e2ffd9
feat(db-test): work on tests
623f4c2
feat(db-test): add role & policy tests, bug fixes
0083436
feat(db-test): fix indentation on JSON inputs
15ffac4
feat(db-test): work on auto migration script
94229d0
feat(db-test): fix travis
8efc52f
feat(db-test): add more migration utilities
c0f4f34
feat(db-test): fix example test
ffa920b
@rudyardrichter
Merge pull request #41 from uc-cdis/feat/db-test
b810ed1 
feat/db-test
feat(db): add docs for resource path formatters
8cdb0f8
@fantix
feat(resource-rules): add yacc
9a00266
@fantix
feat(auth): added auth/request
379445d 
* Also completed auth/proxy
@fantix
chore(indent): simplify code layout
8b25019
@fantix
fix(resource): format path for DB
18df587 
* Also used prepared statement for performance
* Rewrote the lexter to support slashes and quotes
* Faster AND/OR check
@fantix
chore(resource): code review fix
009ef77 
* Added generic lazy prepared statement cache
* Used Request/Response structs in authorize()
@fantix
Merge pull request #40 from uc-cdis/feat/resource-rules
256d491 
PXD-2702: Resource rules yacc
feat(user-api): work on user endpoints & tests
15c7b9d
feat(user-api): add note about migrations
8e56c59
@rudyardrichter
Merge pull request #44 from uc-cdis/feat/user-api
6e07643 
feat(user-api): work on user endpoints & tests
feat(groups): add groups & policies APIs
1ef93c8
feat(auth-resources): work on /auth/resources
3a2cc54
feat(auth-resources): add tests
7074ac8
feat(groups): finish groups api, make crfs
044b51b
feat(groups): add group policy endpoints
aaaf547
@rudyardrichter
Merge pull request #45 from uc-cdis/feat/groups
48a8b6e 
feat(groups): add groups & policies APIs
feat(docs): work on openapi, readme, help
6527f62
@rudyardrichter rudyardrichter merged commit 7c6e2dc into master May 14, 2019
@rudyardrichter rudyardrichter deleted the feat/db branch May 23, 2019 21:33
This was referenced Feb 10, 2020
Merged
Merged
@PlanXCyborg PlanXCyborg mentioned this pull request Feb 18, 2020
Closed
@PlanXCyborg PlanXCyborg mentioned this pull request Mar 2, 2020
Merged
This was referenced Mar 17, 2020
Merged
Closed
Closed
@PlanXCyborg PlanXCyborg mentioned this pull request Apr 16, 2020
Closed
This was referenced May 11, 2020
Closed
Merged
@PlanXCyborg PlanXCyborg mentioned this pull request Oct 21, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fantix fantix fantix left review comments

@Avantol13 Avantol13 Avantol13 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rudyardrichter @fantix @Avantol13