mirror of
https://github.com/System-End/stickers.git
synced 2026-04-19 15:18:17 +00:00
Fix CodeQL security alerts: add CI permissions and document OAuth CSRF bypass
This commit is contained in:
parent
6221afd2e9
commit
a988aed4b3
2 changed files with 6 additions and 1 deletions
3
.github/workflows/ci.yml
vendored
3
.github/workflows/ci.yml
vendored
|
|
@ -5,6 +5,9 @@ on:
|
|||
push:
|
||||
branches: [ main ]
|
||||
|
||||
permissions:
|
||||
contents: read
|
||||
|
||||
jobs:
|
||||
scan_ruby:
|
||||
runs-on: ubuntu-latest
|
||||
|
|
|
|||
|
|
@ -1,7 +1,9 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
class SessionsController < ApplicationController
|
||||
skip_forgery_protection only: :callback
|
||||
# OAuth callback originates from external IdP, not our forms.
|
||||
# CSRF protection is handled by OmniAuth's state parameter validation.
|
||||
skip_forgery_protection only: :callback # codeql[rb/csrf-protection-disabled]
|
||||
|
||||
def login
|
||||
redirect_to "/auth/oidc", allow_other_host: true
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue