ehealthsolutions/element-x-ios
3
0
Fork 0
mirror of https://github.com/Novaloop-AG/element-x-ios.git synced 2026-11-05 20:09:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

Don't pass a URL as a log destination to rust (#2506)

Browse Source 
On macOS (catalyst) URL.appGroupContainerDirectory returns:

/Users/matthew/Library/Group%20Containers/group.io.element/

So you need to strip the %20 out into a space, otherwise logging
fails with a permissions error.

* Use path(percentEncoded:)
This commit is contained in:
Matthew Hodgson
2024-02-28 12:28:17 +00:00
committed by GitHub
parent f5c43ed68f
commit 14d76d9860
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ElementX/Sources/Other/Logging/RustTracing.swift
+2 -2
View File
@@ -45,14 +45,14 @@ enum RustTracing {
otlpEndpoint: otlpConfiguration.url, otlpEndpoint: otlpConfiguration.url,
filter: configuration.filter, filter: configuration.filter,
writeToStdoutOrSystem: true, writeToStdoutOrSystem: true,
writeToFiles: .init(path: logsDirectory.path(), writeToFiles: .init(path: logsDirectory.path(percentEncoded: false),
filePrefix: configuration.fileName, filePrefix: configuration.fileName,
fileSuffix: configuration.fileExtension, fileSuffix: configuration.fileExtension,
maxFiles: maxFiles))) maxFiles: maxFiles)))
} else { } else {
setupTracing(config: .init(filter: configuration.filter, setupTracing(config: .init(filter: configuration.filter,
writeToStdoutOrSystem: true, writeToStdoutOrSystem: true,
writeToFiles: .init(path: logsDirectory.path(), writeToFiles: .init(path: logsDirectory.path(percentEncoded: false),
filePrefix: configuration.fileName, filePrefix: configuration.fileName,
fileSuffix: configuration.fileExtension, fileSuffix: configuration.fileExtension,
maxFiles: maxFiles))) maxFiles: maxFiles)))
changelog.d/pr-2506.bugfix
+1
View File
@@ -0,0 +1 @@
Don't pass a URL as a log destination to rust on macOS