Skip to content

Conversation

@AureliaDolo
Copy link
Contributor

@AureliaDolo AureliaDolo commented Oct 28, 2025

Fix #11360

  • what about timestamp out of ball park ? -> move to certif folder ?
  • test
  • retrieve submit payload
  • poll server for new certificate

@AureliaDolo AureliaDolo force-pushed the aurelia/pki/accept branch 5 times, most recently from fb5e6d3 to b5aa7de Compare November 4, 2025 15:36
- load pki submit local enrollment
@AureliaDolo AureliaDolo marked this pull request as ready for review November 4, 2025 16:02
@AureliaDolo AureliaDolo requested a review from a team as a code owner November 4, 2025 16:02
Comment on lines +91 to +96
let local_file = libparsec_platform_device_loader::get_default_local_pending_file(
&client.config().config_dir,
enrollment_id,
);

let local_pending = load_pki_local_pending(local_file).await?;
Copy link
Contributor

@FirelightFlagboy FirelightFlagboy Nov 5, 2025

Choose a reason for hiding this comment

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

issue (blocking): The accepter won't have access to the local pending save on the submitter filesystem.

The payload is obtain from pki_enroll_list and the value should likely be provided to the function

}
}

pub async fn load_pki_local_pending(
Copy link
Contributor

Choose a reason for hiding this comment

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

issue (blocking): We want need to load a local pending.

Instead we would list all available pending request and at that point we won't need to re-read a request on the filesystem has we would have everything we need extracted from the first step (#11524)

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.

Expose a method in AuthenticatedClient to accept pki request

3 participants