Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • S Storage
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 46
    • Issues 46
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 20
    • Merge requests 20
  • 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
  • System
  • Storage
  • Merge requests
  • !195

Azure || Cosmos Client improvements

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Krishna Nikhil Vedurumudi requested to merge krveduru/goneexp into master May 18, 2021
  • Overview 17
  • Commits 7
  • Pipelines 16
  • Changes 16

Type of change

  • Bug Fix
  • Feature

Please provide link to gitlab issue or ADR(Architecture Decision Record)

Does this introduce a change in the core logic?

  • [YES/NO]

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • GCP
  • IBM

Does this introduce a breaking change?

  • [YES/NO]

What is the current behavior?

What is the new/expected behavior?

Upgrading to latest Azure CoreLib version to make use of optimizations that were done in the usage of Azure Cosmos DB. Includes some test changes that were picked up by OS Core Common upgrade.

Note: Tests have been modified to remove charset=UTF-8 presence in the Content Type header as that has been deprecated in the latest versions of spring

Ref: https://github.com/spring-projects/spring-framework/blob/main/spring-web/src/main/java/org/springframework/http/MediaType.java#L118 Quoting the same

static MediaType - APPLICATION_JSON_UTF8 - Deprecated. as of 5.2 in favor of APPLICATION_JSON since major browsers like Chrome now comply with the specification and interpret correctly UTF-8 special characters without requiring a charset=UTF-8 parameter.

Have you added/updated Unit Tests and Integration Tests?

Any other useful information

Edited May 18, 2021 by Krishna Nikhil Vedurumudi
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: krveduru/goneexp