Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • cern-get-sso-cookie cern-get-sso-cookie
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Jira
    • Jira
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • linuxsupportlinuxsupport
  • rpmsrpms
  • cern-get-sso-cookiecern-get-sso-cookie
  • Merge requests
  • !3

Don't test received cookie with POST when there are no parameters

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Petr Vokac requested to merge vokac/cern-get-sso-cookie:master into master Sep 27, 2019
  • Overview 0
  • Commits 1
  • Pipelines 0
  • Changes 1

One of our website recently configured added behind SSO doesn't like POST request that is used in Auth.pm to test obtained cookie. I added to the test code inspired by https://gitlab.cern.ch/linuxsupport/cern-get-sso-cookie/blob/master/src/WWWCERNSSOAuth.pm#L286 that use GET vs. POST depending on non-empty formdata dictionary. I have no idea if this update works with each CERN SSO web but at least for our website this update works fine (we'll probably get rid of this elog web in near future so please consider if it makes sense to include this update in normal cern-get-sso-cookie package).

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: master