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

Policy Server specs #694

Merged
merged 12 commits into from
Oct 3, 2024
Merged

Policy Server specs #694

merged 12 commits into from
Oct 3, 2024

Conversation

alexcos20
Copy link
Member

@alexcos20 alexcos20 commented Sep 23, 2024 •
edited
Loading

  • Write Policy Server specs - https://github.com/oceanprotocol/ocean-node/blob/feature/policyserver/docs/PolicyServer.md
  • Update ocean-node to check policyServer for newDDO when indexing
  • Update ocean-node to check policyServer for updateDDO when indexing
  • Update ocean-node to check policyServer for intialize handler
  • Update ocean-node to check policyServer for download handler
  • Update ocean-node to check policyServer for encrypt handler
  • Update ocean-node to check policyServer for decrypt handler
  • Add ocean-node policyServer passthrough endpoint

@alexcos20 alexcos20 changed the title Policy Server Policy Server specs Sep 23, 2024
@alexcos20 alexcos20 marked this pull request as ready for review September 30, 2024 18:10
@alexcos20 alexcos20 self-assigned this Oct 1, 2024
@alexcos20 alexcos20 requested a review from valentinbica October 1, 2024 14:43
@alexcos20 alexcos20 linked an issue Oct 1, 2024 that may be closed by this pull request
Define policy server specs + endpoints #682
Closed
jamiehewitt15
jamiehewitt15 reviewed Oct 2, 2024
View reviewed changes
docs/PolicyServer.md Outdated Show resolved Hide resolved
jamiehewitt15
jamiehewitt15 reviewed Oct 2, 2024
View reviewed changes
docs/PolicyServer.md Outdated Show resolved Hide resolved
jamiehewitt15
jamiehewitt15 reviewed Oct 2, 2024
View reviewed changes
docs/PolicyServer.md Outdated Show resolved Hide resolved
alexcos20 and others added 4 commits October 3, 2024 07:17
@alexcos20 @jamiehewitt15
Update docs/PolicyServer.md
839ffc0 
Co-authored-by: Jamie Hewitt <jamie@oceanprotocol.com>
@alexcos20 @jamiehewitt15
Update docs/PolicyServer.md
726eedb 
Co-authored-by: Jamie Hewitt <jamie@oceanprotocol.com>
@alexcos20 @jamiehewitt15
Update docs/PolicyServer.md
7c8292a 
Co-authored-by: Jamie Hewitt <jamie@oceanprotocol.com>
@alexcos20
add try/catch
92692ad
paulo-ocean
paulo-ocean approved these changes Oct 3, 2024
View reviewed changes
Copy link
Contributor

@paulo-ocean paulo-ocean left a comment

Choose a reason for hiding this comment

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

lgtm

jamiehewitt15
jamiehewitt15 approved these changes Oct 3, 2024
View reviewed changes
Copy link
Member

@jamiehewitt15 jamiehewitt15 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@alexcos20 alexcos20 merged commit 9a0bd34 into main Oct 3, 2024
12 checks passed
@alexcos20 alexcos20 deleted the feature/policyserver branch October 3, 2024 15:18
paulo-ocean pushed a commit that referenced this pull request Oct 3, 2024
@alexcos20 @paulo-ocean
Policy Server (#694)
6d55fdb 
Policy server specs and node implementation
This was referenced Nov 22, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paulo-ocean paulo-ocean paulo-ocean approved these changes

@jamiehewitt15 jamiehewitt15 jamiehewitt15 approved these changes

@bogdanfazakas bogdanfazakas Awaiting requested review from bogdanfazakas bogdanfazakas is a code owner

@mariacarmina mariacarmina Awaiting requested review from mariacarmina mariacarmina is a code owner

@valentinbica valentinbica Awaiting requested review from valentinbica

Assignees

@alexcos20 alexcos20

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Define policy server specs + endpoints
3 participants
@alexcos20 @jamiehewitt15 @paulo-ocean