Skip to content

fix: Files list loading with special chars in uid - #759

Closed
Pytal wants to merge 1 commit into
masterfrom
fix/files-dav
Closed

fix: Files list loading with special chars in uid#759
Pytal wants to merge 1 commit into
masterfrom
fix/files-dav

Conversation

@Pytal

@PytalPytal commented Sep 7, 2023

Copy link
Copy Markdown
Contributor

@PytalPytal added type: bug 🐛 Something isn't working 3. to review 3️⃣ Waiting for reviews labels Sep 7, 2023
@Pytal
Pytal requested a review from skjnldsvSeptember 7, 2023 18:22
@PytalPytal self-assigned this Sep 7, 2023
@codecov

codecovBot commented Sep 7, 2023

Copy link
Copy Markdown

Codecov Report

Merging #759 (1335744) into master (2c400db) will decrease coverage by 0.20%.
The diff coverage is 0.00%.

@@ Coverage Diff @@## master #759 +/- ##
==========================================
- Coverage 77.17% 76.98% -0.20% 
==========================================
Files 17 17 Lines 403 404 +1 Branches 108 108 ==========================================
Hits 311 311 - Misses 91 92 +1 
Partials 1 1 
Files ChangedCoverage Δ
lib/dav/dav.ts59.25% <0.00%> (-2.28%)⬇️

📢 Have feedback on the report? Share it here.

@PytalPytal changed the title fix: Files list loading with spaces in uidfix: Files list loading with special chars in uidSep 7, 2023
Signed-off-by: Christopher Ng <chrng8@gmail.com>
Comment threadlib/dav/dav.ts
const client = createClient(remoteURL, {
// Encode valid user id characters
const encodedUrl = encodeURI(remoteURL)
.replaceAll('\'', '%27')

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why replacing the quote here? It is valid for URI according to RFC.

@Pytal

Copy link
Copy Markdown
ContributorAuthor

Superseded by perry-mitchell/webdav-client#360

@PytalPytal closed this Nov 24, 2023
@Pytal
Pytal deleted the fix/files-dav branch November 24, 2023 03:11
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review3️⃣ Waiting for reviewstype: bug🐛 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@Pytal@susnux