Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • S seismic-dms-cpp-lib
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 10
    • Issues 10
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Open Subsurface Data Universe SoftwareOpen Subsurface Data Universe Software
  • Platform
  • Domain Data Management Services
  • Seismic
  • Seismic DMS Suite
  • seismic-dms-cpp-lib
  • Merge requests
  • !34

WIP: minimal set of changes to remove segfault in GCE / GCS (direct or via SD )...

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Anthony Lichnewsky requested to merge slb/al/fix-auth-lack-of-bearer-scheme-on-authorization-headers-and-segfault-everywhere-on-gce into master May 20, 2021
  • Overview 2
  • Commits 1
  • Pipelines 2
  • Changes 10

minimal set of changes to remove segfault in GCE / GCS (direct or via SD ) segfaults, missing Bearer scheme on service calls Authorization headers

direct usage of seismicstore::GcsAccessor ( for example in functionality / regression tests ) on GCE was segfaulting straight away.

usage of GcsAccessor via SDGenericDataset on GCE failed with failures to parse JWT, because API calls are Authorized with the token credentials without the "Bearer" scheme.

For some reason, MRs are accepted without testing on GCE and we managed to distribute libsdapi.so without running regression test. Zenith is broken for now. Omega did not pick it up...

I still need to verify that the change does not break authz on other platforms.

I called this MR "minimal". I'd like to have API changes in the lib/auth directory. these are not meant to be public symbols. But on linux, since they are all in the seismicdrive namespace, they are actually part of the ABI ...

  • getAuthToken() -> getAuthorizationHeader()
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: slb/al/fix-auth-lack-of-bearer-scheme-on-authorization-headers-and-segfault-everywhere-on-gce