Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add supported date format with offset in Nuget repository MetaAnalyzer #1263

Closed
2 tasks done
sahibamittal opened this issue May 17, 2024 · 0 comments · Fixed by #1264
Closed
2 tasks done

Add supported date format with offset in Nuget repository MetaAnalyzer #1263

sahibamittal opened this issue May 17, 2024 · 0 comments · Fixed by #1264
Labels
defect Something isn't working domain/repo-meta-analysis p2 Non-critical bugs, and features that help organizations to identify and reduce risk size/S Small effort

Comments

@sahibamittal
Copy link
Collaborator

Current Behavior

Nuget Analyzer has list of two supported SimpleDateFormat which are allowed for parsing the dates being received from the repository, based on limited test data.

Now, as seen in logs, below date format is also received from the nugget repo. This date has offset dateTime and needs to be added to the list of supported formats in order to avoid the ParseException.

Date example: 1900-01-01T00:00:00+00:00
Format needed: yyyy-MM-dd'T'HH:mm:ss

Proposed Behavior

Add the below format in Nuget analyser to support parsing of offset date time :
yyyy-MM-dd'T'HH:mm:ss

Checklist

@sahibamittal sahibamittal added enhancement New feature or request size/S Small effort domain/repo-meta-analysis defect Something isn't working p2 Non-critical bugs, and features that help organizations to identify and reduce risk and removed enhancement New feature or request labels May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect Something isn't working domain/repo-meta-analysis p2 Non-critical bugs, and features that help organizations to identify and reduce risk size/S Small effort
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant