Skip to content

fix(host-papp): raise SSO request size limit #181

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.

Sign up for GitHub

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

Merged
johnthecat merged 3 commits into from
May 26, 2026
Merged

fix(host-papp): raise SSO request size limit #181

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f911d7d
fix(host-papp): raise SSO request size limit
charlesHetterich May 25, 2026
ac17cb3
test(host-papp): remove SSO request size test
charlesHetterich May 26, 2026
1120480
test(host-papp): remove unused createSession reset
charlesHetterich May 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions packages/host-papp/src/sso/sessionManager/userSession.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@ import type { SigningPayloadRequest, SigningPayloadResponseData, SigningRawReque
// payload is for an SDK version the mobile app doesn't support yet. After
// this timeout the queue task fails, freeing the pool for the next request.
const QUEUE_TASK_TIMEOUT_MS = 180_000;
// Mobile SSO statements allow 256 KiB total; keep headroom for statement/session overhead.
const MAX_SSO_REQUEST_SIZE = 254 * 1024;

function withQueueTimeout<T>(resultAsync: ResultAsync<T, Error>, label: string): ResultAsync<T, Error> {
const timeoutPromise = new Promise<Result<T, Error>>(resolve =>
Expand Down Expand Up @@ -128,6 +130,7 @@ export function createUserSession({
statementStore,
encryption,
prover,
maxRequestSize: MAX_SSO_REQUEST_SIZE,
});

const processedMessages = fieldListView<string>({
Expand Down