MerginMaps / MerginMaps/work-packages

Error raised when project's lock expired before unlocking

Open
#55 0 comments 0 reactions 1 assignee View on GitHub

@ldebek is already working on this.

Since Apr 13, 2023.

bug
Dominant language
Python
Stars
11
Forks
6
PR merge metrics
No merged PRs in 30d

Description

When locking of projects is enabled, and the whole process takes a bit longer to the point that the lock expires (~5 min currently I believe), unlocking in ProjectPadlock.unlock() will fail with a ClientError which is further propagated:

ClientError: The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.

This is because the MM server has removed the transaction, so the URL does not exist anymore.

We should instead just log that the lock has expired and try to continue.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.