Skip to content

Commit

Permalink
Ensure we are not halting the entire application when trying to gener…
Browse files Browse the repository at this point in the history
…ate the debug archive for macOS vfs

Signed-off-by: Claudio Cambra <[email protected]>
  • Loading branch information
claucambra committed Nov 5, 2024
1 parent 7d8fd99 commit 093accc
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/gui/macOS/fileproviderxpc.h
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public slots:
void configureExtensions();
void authenticateExtension(const QString &extensionAccountId) const;
void unauthenticateExtension(const QString &extensionAccountId) const;
void createDebugArchiveForExtension(const QString &extensionAccountId, const QString &filename) const;
void createDebugArchiveForExtension(const QString &extensionAccountId, const QString &filename);

void setFastEnumerationEnabledForExtension(const QString &extensionAccountId, bool enabled) const;

Expand Down
6 changes: 5 additions & 1 deletion src/gui/macOS/fileproviderxpc_mac.mm
Original file line number Diff line number Diff line change
Expand Up @@ -109,9 +109,13 @@
break;
}
}
void FileProviderXPC::createDebugArchiveForExtension(const QString &extensionAccountId, const QString &filename) const
void FileProviderXPC::createDebugArchiveForExtension(const QString &extensionAccountId, const QString &filename)
{
qCInfo(lcFileProviderXPC) << "Creating debug archive for extension" << extensionAccountId << "at" << filename;
if (!fileProviderExtReachable(extensionAccountId)) {
qCWarning(lcFileProviderXPC) << "Extension is not reachable. Cannot create debug archive";
return;
}
// You need to fetch the contents from the extension and then create the archive from the client side.
// The extension is not allowed to ask for permission to write into the file system as it is not a user facing process.
const auto clientCommService = (NSObject<ClientCommunicationProtocol> *)_clientCommServices.value(extensionAccountId);
Expand Down

0 comments on commit 093accc

Please sign in to comment.