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

Bump oCIS commit id #926

Merged
merged 4 commits into from
Apr 11, 2022
Merged

Bump oCIS commit id #926

merged 4 commits into from
Apr 11, 2022

Conversation

pascalwengerter
Copy link
Contributor

Supersedes #900

@pascalwengerter pascalwengerter mentioned this pull request Oct 20, 2021
Closed
@pascalwengerter pascalwengerter force-pushed the 20102021_bump-ocis-commitId branch from 285b8c0 to fa41b85 Compare October 20, 2021 18:18
@wkloucek wkloucek force-pushed the 20102021_bump-ocis-commitId branch 2 times, most recently from f6f2fbd to 323f075 Compare October 21, 2021 06:37
@pascalwengerter pascalwengerter force-pushed the 20102021_bump-ocis-commitId branch from a11c5de to bbdeb2c Compare October 21, 2021 09:55
@pascalwengerter pascalwengerter mentioned this pull request Dec 13, 2021
Closed
@pascalwengerter pascalwengerter force-pushed the 20102021_bump-ocis-commitId branch 2 times, most recently from f3f57da to a7a7a40 Compare March 18, 2022 22:48
@pascalwengerter pascalwengerter force-pushed the 20102021_bump-ocis-commitId branch 2 times, most recently from 3402ac6 to a20fd17 Compare April 6, 2022 09:43
@pascalwengerter pascalwengerter mentioned this pull request Apr 6, 2022
Closed
@pascalwengerter pascalwengerter force-pushed the 20102021_bump-ocis-commitId branch from 8118c79 to eedafd6 Compare April 6, 2022 10:12
@pascalwengerter pascalwengerter mentioned this pull request Apr 6, 2022
Closed
@saw-jan saw-jan mentioned this pull request Apr 6, 2022
Closed
@individual-it individual-it self-assigned this Apr 6, 2022
@individual-it
Copy link
Member

one problem found is that current ocis returns 400 when requested PROPFIND with Depth: 2. This is NOT a bug in ocic because Depth: 2 is not valid, still the older version of ocis and also cC10 are not so fussy, so the issue was not noticed.
For more details see: owncloud/ocis#1975

@individual-it
Copy link
Member

tests don't crash anymore, now need to find out what changed in ocis and adjust the tests

@individual-it individual-it force-pushed the 20102021_bump-ocis-commitId branch from e74102c to 32b0291 Compare April 11, 2022 06:58
pascalwengerter and others added 3 commits April 11, 2022 11:57
@pascalwengerter
Adapt to changed oCIS vars
bf4510e 
generate web assets

clone already checks out the right commit

fix GOPATH in nodejs...

Format starlark

Bump golang CI image to v1.17
@individual-it @pascalwengerter
fix depth header for PROPFIND
5431232 
Remove oCIS workaround

set OCIS_INSECURE when starting ocis

set provider version as tag
@pascalwengerter
bump oCIS commit ID
fd5583c
@pascalwengerter pascalwengerter force-pushed the 20102021_bump-ocis-commitId branch from 8f4b1bb to fd5583c Compare April 11, 2022 09:58
@individual-it individual-it mentioned this pull request Apr 11, 2022
Closed
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

kulmann
kulmann approved these changes Apr 11, 2022
View reviewed changes
Copy link
Contributor

@kulmann kulmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

finally 🤩

@pascalwengerter pascalwengerter merged commit bc371bb into master Apr 11, 2022
@delete-merged-branch delete-merged-branch bot deleted the 20102021_bump-ocis-commitId branch April 11, 2022 15:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kulmann kulmann kulmann approved these changes

Assignees

@individual-it individual-it

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pascalwengerter @individual-it @kulmann