Skip to content
Snippets Groups Projects

Resolve "ClientProtectedResourceMixin allows access if no allowed_scopes are set"

Merged Resolve "ClientProtectedResourceMixin allows access if no allowed_scopes are set"
688-clientprotectedresourcemixin-allows-access-if-no-allowed_scopes-are-set into master
All threads resolved!
Merged Jonathan Weth requested to merge 688-clientprotectedresourcemixin-allows-access-if-no-allowed_scopes-are-set into master
All threads resolved!

Closes #688 (closed)

Edited by Jonathan Weth

Merge request reports

Pipeline #67775 passed with warnings

Stage: prepare
Stage: test
Stage: build
Stage: publish
Stage: docker
Stage: deploy

Pipeline passed with warnings for 480cf497 on 688-clientprotectedresourcemixin-allows-access-if-no-allowed_scopes-are-set

Test coverage 68.00% (0.00%) from 1 job
Approval is optional
Test summary results are being parsed
Full report

Merged by Nik | KlampfradlerNik | Klampfradler 2 years ago (May 3, 2022 10:53pm UTC)

Merge details

Pipeline #67787 passed with warnings

Stage: prepare
Stage: test
Stage: build
Stage: publish
Stage: docker
Stage: deploy
Pipeline: AlekSIS

#67791

    Pipeline passed with warnings for 0d39d5f5 on master

    Test coverage 68.00% (0.00%) from 1 job

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
  • Nik | Klampfradler
    Nik | Klampfradler @nik ·
    Owner

    This needs a regression test.

  • Jonathan Weth added 1 commit

    added 1 commit

    • acf03a29 - Add regression test for broken ClientProtectedResourceMixin

    Compare with previous version

  • Nik | Klampfradler
    Nik | Klampfradler @nik ·
    Owner

    This needs to be backported to 2.7

  • Jonathan Weth resolved all threads

    resolved all threads

  • Jonathan Weth added 1 commit

    added 1 commit

    Compare with previous version

  • Nik | Klampfradler added 10 commits

    added 10 commits

    Compare with previous version

  • Nik | Klampfradler enabled an automatic merge when the pipeline for 480cf497 succeeds

    enabled an automatic merge when the pipeline for 480cf497 succeeds

  • Nik | Klampfradler merged

    merged

  • Nik | Klampfradler mentioned in commit 0d39d5f5

    mentioned in commit 0d39d5f5

    • Please register or sign in to reply