b3nw/gitea-mcp
Archived
1
0
Fork 0
Code Issues 5 Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Silent failure on invalid due_date format in milestone operations #7

New Issue
Open
opened 2025-07-11 15:34:04 -05:00 by b3nw · 0 comments
b3nw commented 2025-07-11 15:34:04 -05:00
Owner
Copy Link

In operation/milestone/milestone.go, the CreateMilestoneFn, EditMilestoneFn, and EditMilestoneByNameFn functions attempt to parse a due_date string. If parsing fails due to an incorrect format, the error is silently ignored. The operation proceeds without setting or updating the due date, and the user receives no feedback that the provided date was invalid.

Recommendation:
The function should return an error if due_date is provided but is in an invalid format.

In `operation/milestone/milestone.go`, the `CreateMilestoneFn`, `EditMilestoneFn`, and `EditMilestoneByNameFn` functions attempt to parse a `due_date` string. If parsing fails due to an incorrect format, the error is silently ignored. The operation proceeds without setting or updating the due date, and the user receives no feedback that the provided date was invalid. **Recommendation:** The function should return an error if `due_date` is provided but is in an invalid format.
b3nw added the bugmilestone-management labels 2025-07-11 15:45:28 -05:00
This repo is archived. You cannot comment on issues.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something isn't working
 enhancement
New feature or request
 feature
New feature implementation
 label-management
Related to label functionality
 milestone-management
Related to milestone functionality
No Label bug milestone-management
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
b3nw
No Assignees
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: b3nw/gitea-mcp#7
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?