Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

We now only encode &, < and > in PROPFIND PCDATA #3240

Merged
merged 1 commit into from
Sep 27, 2022

Conversation

butonic
Copy link
Contributor

@butonic butonic commented Sep 14, 2022

When testing running 100 PROPFIND requests on a collection with 500 folders metrics went from

  • 390.44 KB at 285 ms (lower bound of 100 requests) to
  • 386.47 KB at 269 ms (lower bound of 100 requests)

And we now have clean <d:getetag>"048968ea318d7d0fd23309c598094038"</d:getetag> properties instead of <d:getetag>&#34;048968ea318d7d0fd23309c598094038&#34;</d:getetag>

@butonic butonic self-assigned this Sep 14, 2022
@butonic butonic marked this pull request as ready for review September 14, 2022 20:51
@butonic butonic force-pushed the more-efficient-etag-pcdata branch 2 times, most recently from 65dbf6a to 6d22928 Compare September 21, 2022 11:14
@rhafer rhafer merged commit 3fb6add into cs3org:edge Sep 27, 2022
rhafer added a commit to rhafer/ocis that referenced this pull request Sep 27, 2022
rhafer added a commit to rhafer/ocis that referenced this pull request Sep 27, 2022
butonic pushed a commit to owncloud/ocis that referenced this pull request Sep 27, 2022
ownclouders pushed a commit to owncloud/ocis that referenced this pull request Sep 27, 2022
Author: Ralf Haferkamp <[email protected]>
Date:   Tue Sep 27 20:38:08 2022 +0200

    Bump reva (#4694)

    To get cs3org/reva#3240
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants