-
Notifications
You must be signed in to change notification settings - Fork 8
Feat/dpop worker thread support #33
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.
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
Open
kkalass
wants to merge
14
commits into
anusii:dev
Choose a base branch
from
kkalass:feat/dpop-worker-thread-support
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
4375724
feat: migrate from custom OpenID implementation to oidc library
kkalass 4c398cd
Added API Documentation
kkalass b39c8b9
Updated README
kkalass 0f1acec
Added some more entries to .gitignore
kkalass 7a43e59
fix: enable refresh tokens by automatically adding consent prompt forβ¦
kkalass 2b93eba
feat: enable strict JWT verification by default for enhanced security
kkalass 14b9eef
Fix for refreshes: make sure we always have headers
kkalass 0333efd
docs: add network permissions setup for Android and macOS
kkalass b0e291c
feat: add worker thread support for DPoP token generation
kkalass 8258c6d
refactor: eliminate fast_rsa dependency from serializable types
kkalass 8b158b3
refactor: create Flutter-free entry point for worker threads
kkalass a0b52c9
upgraded dependencies
kkalass 721a82a
Upgraded dependencies
kkalass abf70db
further dependency updates
kkalass File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,91 @@ | ||
| name: Deploy Flutter Web to GitHub Pages | ||
|
|
||
| on: | ||
| push: | ||
| # FIXME: Change to [ main ] when merging to upstream - this is currently set for fork development | ||
| branches: [ feat/migrate-to-bdaya-oidc-security-fix ] | ||
| # FIXME: Add back pull_request trigger when merging to upstream: | ||
| # pull_request: | ||
| # branches: [ main ] | ||
|
|
||
| # Sets permissions of the GITHUB_TOKEN to allow deployment to GitHub Pages | ||
| permissions: | ||
| contents: read | ||
| pages: write | ||
| id-token: write | ||
|
|
||
| # Allow only one concurrent deployment, skipping runs queued between the run in-progress and latest queued. | ||
| # However, do NOT cancel in-progress runs as we want to allow these production deployments to complete. | ||
| concurrency: | ||
| group: "pages" | ||
| cancel-in-progress: false | ||
|
|
||
| jobs: | ||
| build: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Setup Flutter | ||
| uses: subosito/flutter-action@v2 | ||
| with: | ||
| channel: 'stable' | ||
| cache: true | ||
|
|
||
| - name: Disable Flutter analytics | ||
| run: flutter config --no-analytics | ||
|
|
||
| - name: Cache pub dependencies | ||
| uses: actions/cache@v4 | ||
| with: | ||
| path: ~/.pub-cache | ||
| key: ${{ runner.os }}-pub-${{ hashFiles('**/pubspec.lock') }} | ||
| restore-keys: ${{ runner.os }}-pub- | ||
|
|
||
| - name: Install dependencies (root) | ||
| run: flutter pub get | ||
|
|
||
| - name: Install dependencies (example) | ||
| run: | | ||
| cd example | ||
| flutter pub get | ||
|
|
||
| - name: Build web app | ||
| run: | | ||
| cd example | ||
| flutter build web --release --base-href /solid_auth/example/ | ||
|
|
||
| - name: Setup Pages | ||
| uses: actions/configure-pages@v4 | ||
|
|
||
| - name: Prepare deployment directory | ||
| run: | | ||
| # Create a deployment directory | ||
| mkdir -p deploy/example | ||
|
|
||
| # Copy Flutter web build (built in example directory) | ||
| cp -r example/build/web/* deploy/example/ | ||
|
|
||
| # Copy client profile and redirect files | ||
| cp example/client-profile.jsonld deploy/example/ | ||
| cp example/redirect.html deploy/example/ | ||
|
|
||
|
|
||
| - name: Upload artifact | ||
| uses: actions/upload-pages-artifact@v3 | ||
| with: | ||
| path: ./deploy | ||
|
|
||
| deploy: | ||
| environment: | ||
| name: github-pages | ||
| url: ${{ steps.deployment.outputs.page_url }} | ||
| runs-on: ubuntu-latest | ||
| needs: build | ||
| # FIXME: Change to `if: github.ref == 'refs/heads/main'` when merging to upstream | ||
| if: github.ref == 'refs/heads/feat/migrate-to-bdaya-oidc-security-fix' | ||
| steps: | ||
| - name: Deploy to GitHub Pages | ||
| id: deployment | ||
| uses: actions/deploy-pages@v4 | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This workflow is currently hardcoded to run only on the
feat/migrate-to-bdaya-oidc-security-fixbranch (and the deploy job checks the same ref). If this is intended for upstream/mainline usage, switch triggers/conditions tomain(or make them configurable) before merging; otherwise the workflow will never run for the default branch.