Skip to content

Conversation

@bmwill
Copy link
Contributor

@bmwill bmwill commented Oct 3, 2024

Description

Describe the changes or additions included in this PR.

Test plan

How did you test the new or updated feature?


Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

  • Protocol:
  • Nodes (Validators and Full nodes):
  • Indexer:
  • JSON-RPC:
  • GraphQL:
  • CLI:
  • Rust SDK:
  • REST API:

@bmwill bmwill requested a review from sadhansood October 3, 2024 22:41
@vercel
Copy link

vercel bot commented Oct 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sui-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 4, 2024 7:37pm
3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
multisig-toolkit ⬜️ Ignored (Inspect) Visit Preview Oct 4, 2024 7:37pm
sui-kiosk ⬜️ Ignored (Inspect) Visit Preview Oct 4, 2024 7:37pm
sui-typescript-docs ⬜️ Ignored (Inspect) Visit Preview Oct 4, 2024 7:37pm

Copy link
Contributor

@sadhansood sadhansood left a comment

Choose a reason for hiding this comment

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

Thanks @bmwill, this looks great. And thanks for adding the list full checkpoint as well. I didn't see it but is it possible to add an endpoint which tells user whether unstable endpoints are enabled on the server.

@bmwill
Copy link
Contributor Author

bmwill commented Oct 4, 2024

I didn't see it but is it possible to add an endpoint which tells user whether unstable endpoints are enabled on the server.

No and i don't know if we should (although you can technically get this information via the openapi spec/explorer)

@sadhansood
Copy link
Contributor

I didn't see it but is it possible to add an endpoint which tells user whether unstable endpoints are enabled on the server.

No and i don't know if we should (although you can technically get this information via the openapi spec/explorer)

Makes sense!

bmwill added 6 commits October 4, 2024 14:35
Enable marking APIs as either stable or unstable. By default unstable
APIs are disabled and not served but the `enable_unstable_apis` can be
set to `true` in order to enable serving of unstable APIs.

Example enabling enable_unstable_apis in fullnode.yaml:

```yaml
enable_experimental_rest_api: true
rest:
  enable_unstable_apis: true
```
Stabalize the get and list checkpoints apis as well as introduce a new
list full checkpoints api.
Until the rest service as a whole is "stabalized" with a sane set of
default stable apis, have the default be to enable all unstable apis.
@bmwill bmwill force-pushed the rest-stabilization branch from 2e2aa1a to 75528ae Compare October 4, 2024 19:35
@bmwill bmwill merged commit 3b665eb into MystenLabs:main Oct 4, 2024
45 checks passed
@bmwill bmwill deleted the rest-stabilization branch October 4, 2024 20:18
jkrvivian added a commit to iotaledger/iota that referenced this pull request Jun 6, 2025
)

# Description of change

- Upstream range: [v1.35.4, v1.36.2)
- Port the following Sui's commits which are contained in the single PR
MystenLabs/sui#19701
-
MystenLabs/sui@512a4e0
-
MystenLabs/sui@7d05486
-
MystenLabs/sui@ea81ea5
-
MystenLabs/sui@591f96c
-
MystenLabs/sui@13c1ec7
-
MystenLabs/sui@3b665eb
- Descriptions from commits
- Enable marking APIs as either stable or unstable. By default unstable
APIs are disabled and not served but the `enable_unstable_apis` can be
set to `true` in order to enable serving of unstable APIs.
- Stabalize the get and list checkpoints apis as well as introduce a new
list full checkpoints api.
- Until the rest service as a whole is "stabalized" with a sane set of
default stable apis, have the default be to enable all unstable apis.
- Notes
- The `enable_experimental_rest_api` was renamed to be `enable_rest_api`
in #3740
- The `worker_public_key` was renamed to be `protocol_public_key` in
#3151
- Because the `iota-sdk-types` does not exit, remain original
`iota-sdk2` instead
- The `proc-macro2` version is updated in the upstream change, hence
here we update it to be the latest version accordingly. (see
https://crates.io/crates/proc-macro2)


## Links to any relevant issues

Part of #3990 

## Type of change

- Enhancement (a non-breaking change which adds functionality)

## How the change has been tested

Ran the local network with `RUST_LOG=info cargo run --release --bin iota
start --force-regenesis --with-faucet`

---------

Co-authored-by: jkrvivian <[email protected]>
muXxer pushed a commit to iotaledger/iota that referenced this pull request Jun 10, 2025
)

# Description of change

- Upstream range: [v1.35.4, v1.36.2)
- Port the following Sui's commits which are contained in the single PR
MystenLabs/sui#19701
-
MystenLabs/sui@512a4e0
-
MystenLabs/sui@7d05486
-
MystenLabs/sui@ea81ea5
-
MystenLabs/sui@591f96c
-
MystenLabs/sui@13c1ec7
-
MystenLabs/sui@3b665eb
- Descriptions from commits
- Enable marking APIs as either stable or unstable. By default unstable
APIs are disabled and not served but the `enable_unstable_apis` can be
set to `true` in order to enable serving of unstable APIs.
- Stabalize the get and list checkpoints apis as well as introduce a new
list full checkpoints api.
- Until the rest service as a whole is "stabalized" with a sane set of
default stable apis, have the default be to enable all unstable apis.
- Notes
- The `enable_experimental_rest_api` was renamed to be `enable_rest_api`
in #3740
- The `worker_public_key` was renamed to be `protocol_public_key` in
#3151
- Because the `iota-sdk-types` does not exit, remain original
`iota-sdk2` instead
- The `proc-macro2` version is updated in the upstream change, hence
here we update it to be the latest version accordingly. (see
https://crates.io/crates/proc-macro2)


## Links to any relevant issues

Part of #3990 

## Type of change

- Enhancement (a non-breaking change which adds functionality)

## How the change has been tested

Ran the local network with `RUST_LOG=info cargo run --release --bin iota
start --force-regenesis --with-faucet`

---------

Co-authored-by: jkrvivian <[email protected]>
filipdulic pushed a commit to iotaledger/iota that referenced this pull request Jul 16, 2025
)

# Description of change

- Upstream range: [v1.35.4, v1.36.2)
- Port the following Sui's commits which are contained in the single PR
MystenLabs/sui#19701
-
MystenLabs/sui@512a4e0
-
MystenLabs/sui@7d05486
-
MystenLabs/sui@ea81ea5
-
MystenLabs/sui@591f96c
-
MystenLabs/sui@13c1ec7
-
MystenLabs/sui@3b665eb
- Descriptions from commits
- Enable marking APIs as either stable or unstable. By default unstable
APIs are disabled and not served but the `enable_unstable_apis` can be
set to `true` in order to enable serving of unstable APIs.
- Stabalize the get and list checkpoints apis as well as introduce a new
list full checkpoints api.
- Until the rest service as a whole is "stabalized" with a sane set of
default stable apis, have the default be to enable all unstable apis.
- Notes
- The `enable_experimental_rest_api` was renamed to be `enable_rest_api`
in #3740
- The `worker_public_key` was renamed to be `protocol_public_key` in
#3151
- Because the `iota-sdk-types` does not exit, remain original
`iota-sdk2` instead
- The `proc-macro2` version is updated in the upstream change, hence
here we update it to be the latest version accordingly. (see
https://crates.io/crates/proc-macro2)


## Links to any relevant issues

Part of #3990 

## Type of change

- Enhancement (a non-breaking change which adds functionality)

## How the change has been tested

Ran the local network with `RUST_LOG=info cargo run --release --bin iota
start --force-regenesis --with-faucet`

---------

Co-authored-by: jkrvivian <[email protected]>
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