Skip to content

Improving security around executor and various bug fixes#8

Merged
abidlabs merged 25 commits intomainfrom
executor-security-login
Jan 29, 2026
Merged

Improving security around executor and various bug fixes#8
abidlabs merged 25 commits intomainfrom
executor-security-login

Conversation

@abidlabs
Copy link
Member

@abidlabs abidlabs commented Jan 28, 2026 •
edited
Loading

A bunch of bug fixes in this PR

  • Previously, all users shared a single executor with shared results cache, HF tokens, and Gradio clients—allowing potential cross-user data leakage. Now each connection gets its own ExecutionSession with isolated state.
  • Additionally, FnNodes now execute sequentially by default (one at a time per session) to prevent resource contention, with opt-in concurrent execution via concurrent=True and support for concurrency_group to share limits between nodes. GradioNode and InferenceNode continue to run concurrently as they're external API calls. The README has been updated with documentation on concurrency behavior and recommendations to prefer GradioNode/InferenceNode over FnNode when possible

@abidlabs
Copy link
Member Author

abidlabs commented Jan 28, 2026 •
edited
Loading

🦄 change detected

This Pull Request includes changes to the following packages.

Package Version
daggr minor
  • Improving security around executor and various bug fixes

‼️ Changeset not approved. Ensure the version bump is appropriate for all packages before approving.

  • Maintainers can approve the changeset by checking this checkbox.

Something isn't right?
  • Maintainers can change the version label to modify the version bump.
  • If the bot has failed to detect any changes, or if this pull request needs to update multiple packages to different versions or requires a more comprehensive changelog entry, maintainers can update the changelog file directly.

@abidlabs
Copy link
Member Author

abidlabs commented Jan 28, 2026 •
edited
Loading

🪼 branch checks and previews

• Name Status URL
🦄 Changes detected! Details

@abidlabs abidlabs changed the title Improving security around executor and other aspects Improving security around executor and various bug fixes Jan 28, 2026
@abidlabs abidlabs marked this pull request as draft January 28, 2026 21:32
@abidlabs abidlabs marked this pull request as ready for review January 29, 2026 07:52
@abidlabs abidlabs merged commit 53512dd into main Jan 29, 2026
4 checks passed
@abidlabs abidlabs mentioned this pull request Jan 29, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@abidlabs @gradio-pr-bot