Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • gst-plugins-base gst-plugins-base
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 641
    • Issues 641
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 80
    • Merge requests 80
  • 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
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GStreamerGStreamer
  • gst-plugins-basegst-plugins-base
  • Merge requests
  • !53

rtspconnection: Replace Auth header instead of append

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Per Forlin requested to merge perfn/gst-plugins-base:fix-multiple-auth-headers into master Dec 03, 2018
  • Overview 7
  • Commits 1
  • Pipelines 2
  • Changes 1

gst_rtsp_connection_send() adds the Authorization header to the request. If this function is being called multiple times with the same request it will add one more Authorization header every time.

To fix to this issue do not append a new Authorization header on top of an existing ones. Remove any existing Authorization headers first and then add the new one.

Fixes gst-plugins-good#425 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix-multiple-auth-headers