Monitors and federation server side #225

calvinrp · 2023-11-08T19:22:24Z

Implements the server-side for the following issues:

Deferred #229 for a follow up PR, due to the large diff.

calvinrp · 2023-11-09T13:18:57Z

TODO:

more tests
validation on package publish for operator log defined namespaces
adding new permissions for defining and importing namespaces

The client-side will largely be done in a separate PR. Also, I'll have a separate PR for #229 , to avoid a really messy diff in this PR.

But helpful to start reviewing...

crates/api/src/v1/monitor.rs

crates/protocol/src/operator/state.rs

…aintain case for presentation and code gen

crates/server/src/api/v1/monitor.rs

lann

Partial review covering the api crate changes

crates/api/src/v1/ledger.rs

crates/api/src/v1/monitor.rs

calvinrp · 2023-11-13T21:55:13Z

Refactored the verify checkpoint endpoint. The OpenAPI rendered spec

crates/api/src/v1/monitor.rs

…nd fetch package names

…e case sensitive but are unique in a case sensitive way

…dy for publishing packages

crates/api/src/v1/monitor.rs

crates/protocol/src/operator/state.rs

crates/protocol/tests/operator-logs/longer.json

crates/server/src/api/v1/mod.rs

crates/server/src/api/v1/monitor.rs

…s a `501` error

calvinrp and others added 3 commits November 6, 2023 14:57

added monitor verify checkpoint, get ledger, and fetch package IDs

80889f2

added operator entries for defined and imported namespaces

595c44c

updated openapi.yaml

2f5dcc4

calvinrp requested review from lann and esoterra November 8, 2023 19:22

calvinrp added 3 commits November 8, 2023 15:09

added validation errors for publishing packages to namespaces

1102295

renamed fetch endpoint to /fetch/names and fixed a bug

247363b

fixed bug in debug endpoint

2f51a7a

calvinrp added 3 commits November 9, 2023 10:06

added operator permissions for defining and importing namespaces

b34824b

cargo fmt

d57e7de

added validation on package publish for operator log defined namespaces

1cdd9ea

esoterra reviewed Nov 9, 2023

View reviewed changes

crates/api/src/v1/monitor.rs Outdated Show resolved Hide resolved

crates/protocol/src/operator/state.rs Outdated Show resolved Hide resolved

calvinrp added 5 commits November 9, 2023 22:55

renamed to NamespaceState

331bb06

revised checkpoint verify endpoint for monitors and registries

989dc8f

package name and namespace are unique in a case insensitive way but m…

ea5d9aa

…aintain case for presentation and code gen

renamed methods for verifying package can be published to namespace

65de51d

revised names

de67886

calvinrp commented Nov 12, 2023

View reviewed changes

crates/server/src/api/v1/monitor.rs Outdated Show resolved Hide resolved

lann reviewed Nov 13, 2023

View reviewed changes

calvinrp added 2 commits November 13, 2023 12:25

changed the contentType for the ledger content sources

64078b3

refactored monitor verify checkpoint api endpoint

a0504b6

fixed comment

8ab36c0

lann reviewed Nov 14, 2023

View reviewed changes

crates/api/src/v1/monitor.rs Outdated Show resolved Hide resolved

crates/api/src/v1/monitor.rs Outdated Show resolved Hide resolved

crates/api/src/v1/monitor.rs Outdated Show resolved Hide resolved

calvinrp added 5 commits November 14, 2023 13:46

adding the client api methods for verify checkpoint, ledger sources a…

a6e1047

…nd fetch package names

revised verify checkpoint endpoint

a45e149

reworded openapi.yaml

21520a3

openapi example changed

d2b98a2

implements bytecodealliance#232 where package names and namespaces ar…

bce7caf

…e case sensitive but are unique in a case sensitive way

calvinrp added 3 commits November 15, 2023 20:51

added tests for fetch package names

b614fd0

added alias packageName for the packageId field in the request bo…

0feef4f

…dy for publishing packages

added test for ledger sources endpoint

fe8ea9b

esoterra reviewed Nov 16, 2023

View reviewed changes

calvinrp added 3 commits November 16, 2023 12:03

added comment to registry header extractor to indicate that it return…

20e4313

…s a `501` error

revised namespace validation

411a9db

revised monitor api checkpoint verification

e76db8d

esoterra approved these changes Nov 16, 2023

View reviewed changes

calvinrp merged commit a233eb4 into bytecodealliance:main Nov 16, 2023
6 checks passed

calvinrp deleted the monitors-and-federation-server-side branch November 16, 2023 21:21

calvinrp mentioned this pull request Nov 16, 2023

Add Namespace Definition/Imports to Operator Log #224

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Monitors and federation server side #225

Monitors and federation server side #225

calvinrp commented Nov 8, 2023 •

edited

Loading

calvinrp commented Nov 9, 2023 •

edited

Loading

lann left a comment

calvinrp commented Nov 13, 2023

Monitors and federation server side #225

Monitors and federation server side #225

Conversation

calvinrp commented Nov 8, 2023 • edited Loading

calvinrp commented Nov 9, 2023 • edited Loading

lann left a comment

Choose a reason for hiding this comment

calvinrp commented Nov 13, 2023

calvinrp commented Nov 8, 2023 •

edited

Loading

calvinrp commented Nov 9, 2023 •

edited

Loading