Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

General: Cloud mongo ca certificate issue #2095

Merged
merged 19 commits into from
Oct 4, 2021

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Oct 1, 2021

Issue

Since 30.9.2021 does not work cloud mongo connection correctly because of invalid ssl certificates on most of machines. It is issue mainly on Windows machines.

Changes

  • add certifi certificate to creation of mongo connection if mongodb+srv scheme is used or when ssl=true or tls=true is in mongo url query
  • add session creation to validation of mongo connection
    • server info does not trigger any mongo server queries which are important to validate
  • removed decompose/compose function for mongo parsing from igniter (which removes possibly important attributes)
  • fixed import order bug caused by Added running configurable disk mapping command before start of OP #2091 (comment)
  • validation and mongo connection inside openpype happens on single place through OpenPypeMongoConnection class
    • avalon started to use the same method to create connection too
PR depends on
avalon-core ynput/avalon-core#373

@iLLiCiTiT iLLiCiTiT self-assigned this Oct 1, 2021
@iLLiCiTiT iLLiCiTiT added severity: HIGH type: bug Something isn't working labels Oct 1, 2021
@iLLiCiTiT iLLiCiTiT changed the title General: Cliud mongo ca certificate issue General: Cloud mongo ca certificate issue Oct 1, 2021
@64qam
Copy link
Member

64qam commented Oct 1, 2021

Running from code works well. However, a new build needs to be tuned (after a new environment was created).

image (21)
..

@64qam
Copy link
Member

64qam commented Oct 1, 2021

Works if /admin?ssl=true is appended to the current mongo string

@iLLiCiTiT iLLiCiTiT merged commit 7d837e3 into develop Oct 4, 2021
@iLLiCiTiT iLLiCiTiT deleted the bugfix/mongo_ca_certificate branch December 13, 2021 15:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants