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.

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

feat(device-manager): Allow unsigned ACAPs #122

Merged
merged 3 commits into from
Oct 31, 2024

Conversation

kjughx
Copy link
Contributor

@kjughx kjughx commented Oct 31, 2024

For development, it is not feasible to sign the ACAP for every iteration.

Allow unsigned ACAPs as part of the initialization process.

Checklist before requesting a review

  • I have performed a self-review of my own code
  • I have verified that the code builds perfectly fine on my local system
  • I have added tests that prove my fix is effective or that my feature works
  • I have commented my code, particularly in hard-to-understand areas
  • I have verified that my code follows the style already available in the repository
  • I have made corresponding changes to the documentation

For development, it is not feasible to sign the ACAP for every
iteration.

Allow unsigned ACAPs as part of the initialization process.
@kjughx kjughx requested a review from a team as a code owner October 31, 2024 16:44
- Use the term "ACAP application" because this is the replacement used
  in PR267 of AxisCommunications/acap-native-sdk-examples making me
  think it is the preferred term.
- Make no attempt to check if the request error is expected for the
  current firmware for simplicity. This follows the example established
  with `restore_root_ssh_user`.
- Deviate from the aforementioned pattern by naming the firmware in
  which the API was introduced because this seems like it could be
  useful information, it is unlikely to become incorrect and earlier
  commits don't give a reason for omitting this information.
@apljungquist apljungquist merged commit 9603842 into AxisCommunications:main Oct 31, 2024
2 checks passed
apljungquist added a commit that referenced this pull request Nov 5, 2024
* upstream/main: (48 commits)
  chore: Use workspace dependencies consistently (#126)
  fix(axevent_example): Return `Break` from signal handlers (#125)
  chore(deps): bump ghcr.io/devcontainers/features/common-utils (#124)
  feat(device-manager): Allow unsigned ACAPs (#122)
  feat(acap-build): Enable bypassing `acap-build` (#119)
  chore: streamline CI (#118)
  fix: Don't invalidate Cargo caches (#117)
  fix(acap-ssh-utils): Warn users that commands may break (#114)
  feat: Factor out `acap-build` wrapper to lib crate (#112)
  feat: Add `fleet-manager` program (#111)
  feat: Add edge storage wrapper and example (#108)
  fix: Measure apparent size (#107)
  chore(acap-logging): Bump version (#106)
  fix(cargo-acap-sdk): Set executable bits (#105)
  fix(acap-logging)!: Disable default features for env_logger (#103)
  chore: Track size of apps (#102)
  chore: Don't use clearly defined for license info (#101)
  Update rust toolchain to 1.80.1 (#100)
  fix!: Make builds reproducible in dev-container (#99)
  feat(axevent)!: Use generic callbacks and remove leak (#59)
  ...
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