vinzenz/hyperhive
0
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: add read:notification scope, always remint token on ensure_user_for

Browse source
This commit is contained in:
damocles 2026-05-20 19:05:25 +02:00
parent 3e5d430873
commit 72aef13eaf
1 changed files with 8 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

17
hive-c0re/src/forge.rs
View file

@ -53,8 +53,10 @@ const SEEDED_ORGS: &[&str] = &[CONFIG_ORG];
/// a forge namespace). /// a forge namespace).
/// - `read:user` — token-owner endpoint clients call to introspect. /// - `read:user` — token-owner endpoint clients call to introspect.
/// - `write:misc` — hooks, attachments, the rest of the long tail. /// - `write:misc` — hooks, attachments, the rest of the long tail.
/// - `read:notification` — required by forge_notify to poll
/// `GET /notifications` for unread PR/review events.
const TOKEN_SCOPES: &str = const TOKEN_SCOPES: &str =
"read:user,write:user,write:repository,write:issue,write:organization,write:misc"; "read:user,write:user,read:notification,write:repository,write:issue,write:organization,write:misc";
/// Token file inside the agent's bind-mounted state dir (visible as /// Token file inside the agent's bind-mounted state dir (visible as
/// `/state/forge-token` from inside the container). /// `/state/forge-token` from inside the container).
@ -226,19 +228,16 @@ async fn mint_and_persist_token(name: &str, path: &Path) -> Result<()> {
Ok(()) Ok(())
} }
/// Ensure `name` has a forgejo user + token file. No-op when the /// Ensure `name` has a forgejo user + token file. Always re-mints the
/// token file is already present. Safe to call on every spawn and /// token so the on-disk file always reflects the current `TOKEN_SCOPES`.
/// on every hive-c0re startup. /// Safe to call on every spawn and on every hive-c0re startup.
pub async fn ensure_user_for(name: &str) -> Result<()> { pub async fn ensure_user_for(name: &str) -> Result<()> {
if !is_present().await { if !is_present().await {
return Ok(()); return Ok(());
} }
let path = token_path(name);
if path.exists() {
return Ok(());
}
ensure_user_exists(name, false).await?; ensure_user_exists(name, false).await?;
mint_and_persist_token(name, &path).await ensure_user_email(name).await;
mint_and_persist_token(name, &token_path(name)).await
} }
/// Ensure the bootstrap `core` admin user + a token at /// Ensure the bootstrap `core` admin user + a token at