Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

[BUG] wash claims sign allows empty versions and revisions #958

Closed

Conversation

ahmedtadde
Copy link
Contributor

@ahmedtadde ahmedtadde commented Oct 30, 2023

Feature or Problem

Currently, when signing an actor with wash, the version (--ver) and revision (--rev) args are not required. This will lead to claims that are missing some important information.

Related Issues

wasmCloud/wasmCloud#861

Release Information

v0.21.1

Consumer Impact

stricter wash claims sign cmd that ultimately improves the metadata output of wash inspect

Testing

Built on platform(s)

  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
  • x86_64-windows

Tested on platform(s)

  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
  • x86_64-windows

Acceptance or Integration

updated integration tests integration_claims_sign, integration_claims_call_alias

@ahmedtadde ahmedtadde marked this pull request as ready for review October 30, 2023 17:56
@ahmedtadde ahmedtadde requested a review from a user October 30, 2023 17:56
@connorsmith256
Copy link
Contributor

Hey @ahmedtadde, as of today, wash has been moved to the wasmCloud repo 🎉

Would you mind re-opening these changes against the main repo? I'm working on switching this repo to be a public archive

@ahmedtadde ahmedtadde changed the title [BUG] wash claims sign allows empty versions and revisions #859 [BUG] wash claims sign allows empty versions and revisions Oct 30, 2023
@ahmedtadde ahmedtadde closed this Oct 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants