n8n - 💡(How to fix) Fix Issue listing 'Version History' since 2.17.5 upgrade [1 comments, 2 participants]

Describe the problem/error/question

We're seeing a new error when non-admin users try to list the Version History on their workflows.

What is the error message (if any)?

When a user opens their workflow and tries to view Version History they see an error popup that states:

• error: all

TL;DR

The error is likely due to a recent commit that introduced a permission check, and a quick fix would be to revert or modify this change to allow non-admin users to list version history.

Guidance

• Review the recent commit 9dfe97f8bb00f14437bc013b9ca7047804f3d70d and assess the impact of the permission check on non-admin users.
• Consider modifying the commit to add an exception for version history listing, allowing non-admin users to access this feature.
• Test the modified code to ensure it resolves the issue without introducing security vulnerabilities.
• As a temporary workaround, consider reverting to version 2.17.3, as the issue is not present in this version.

Example

No code snippet is provided, as the issue is related to a specific commit and requires a review of the code changes.

Notes

The issue is specific to version 2.17.5, and the commit in question has introduced a permission check that is causing the error. The fix will depend on the specific requirements and constraints of the application.

Recommendation

Apply a workaround by modifying the recent commit to allow non-admin users to list version history, as this will address the critical security issues fixed in 2.17.5 while resolving the current issue.