Skip to content

Deploy challenge info to frontend #81

Open
detjensrobert wants to merge 27 commits into
mainfrom
dr/frontend-push
Open

Deploy challenge info to frontend #81
detjensrobert wants to merge 27 commits into
mainfrom
dr/frontend-push

Conversation

@detjensrobert

@detjensrobert detjensrobert commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

@cacama-valvata has provided the schema for what frontend wants for challenge state. This implements that instead of the temporary markdown writeout that we hacked in last year.

This also includes some temporary hacky updates to the challenge difficulty field so that it now matches the frontend difficulty config that was moved to be a string instead of an int. This is hacky and the parsing/validation should happen at parse time instead of deploy time but this is enough to make this functional and will be cleaned up as part of #71 anyway.

Resolves #62.
Resolves #13.

@detjensrobert detjensrobert changed the title remove temporary markdown writeout Deploy challenge info to frontend Jan 29, 2026
Comment thread src/deploy/frontend.rs Outdated
@detjensrobert
detjensrobert force-pushed the dr/frontend-push branch 3 times, most recently from b8028e4 to e0dc239 Compare February 11, 2026 06:58
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Replaces the temporary challenge info markdown output. This currently
doesn't print out the response from frontend.

Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
@cacama-valvata

Copy link
Copy Markdown
Contributor

keeping open until frontend connector to backend is finished

Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
This struct is only used to validate the structure of the response from
Frontend, and is intentionally not used otherwise.

Signed-off-by: Robert Detjens <github@detjens.dev>
detjensrobert and others added 5 commits May 23, 2026 18:12
Co-authored-by: Kai Morita-McVey <kai@moritamcvey.com>
Signed-off-by: Robert Detjens <github@detjens.dev>
Co-authored-by: Kai Morita-McVey <kai@moritamcvey.com>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Signed-off-by: Robert Detjens <github@detjens.dev>
Use system CA trust via platform-verifier

@KekoaM KekoaM left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Overall looks good to me. Two nits but they do not impact functionality.

Comment thread src/configparser/challenge.rs Outdated
Comment thread src/commands/validate.rs
}
}

info!(" config ok!");

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why the leading space?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The validate commands indent their OK for clarity:
image

Signed-off-by: Robert Detjens <github@detjens.dev>
@detjensrobert
detjensrobert requested a review from KekoaM May 26, 2026 03:25

@KekoaM KekoaM left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lgtm

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.

Connect to frontend when ready implement check-access for RCTF frontend

3 participants