Skip to content

Extend ntsketest to the full KE->NTPv4 round-trip#568

Closed
sofiascalzo wants to merge 1 commit into
facebook:mainfrom
sofiascalzo:export-D113441692
Closed

Extend ntsketest to the full KE->NTPv4 round-trip#568
sofiascalzo wants to merge 1 commit into
facebook:mainfrom
sofiascalzo:export-D113441692

Conversation

@sofiascalzo

Copy link
Copy Markdown

Summary:
Extended smoke to prove the whole authenticated exchange works end to end. This extends it to run the NTPv4 phase after KE

build a signed request, send it, and verify the response

reusing the shared client helpers so the tool stays a thin driver. A server-side test also drives the real request path with those public helpers.

Reviewed By: vvfedorenko

Differential Revision: D113441692

Summary:
Extended smoke to prove the whole authenticated exchange works end to end. This extends it to run the NTPv4 phase after KE 

build a signed request, send it, and verify the response 

reusing the shared client helpers so the tool stays a thin driver. A server-side test also drives the real request path with those public helpers.

Reviewed By: vvfedorenko

Differential Revision: D113441692
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 24, 2026
@meta-codesync

meta-codesync Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

@sofiascalzo has exported this pull request. If you are a Meta employee, you can view the originating Diff in D113441692.

@meta-codesync

meta-codesync Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

This pull request has been merged in b2dde4c.

@meta-codesync meta-codesync Bot added the Merged label Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant