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

ProfileParameters in TO API uses non-RFC3339 date/time strings #7737

Closed
jagan-parthiban opened this issue Aug 22, 2023 · 0 comments · Fixed by #7738
Closed

ProfileParameters in TO API uses non-RFC3339 date/time strings #7737

jagan-parthiban opened this issue Aug 22, 2023 · 0 comments · Fixed by #7738
Labels
low impact affects only a small portion of a CDN, and cannot itself break one tech debt rework due to choosing easy/limited solution Traffic Ops related to Traffic Ops

Comments

@jagan-parthiban
Copy link
Contributor

Parent Issue: #5911

Breaking down the parent issue #5911 in smaller issues to raise separate, reviewable PRs for each of the services.

Traffic Control components affected:

  • Traffic Ops
  • Documentation

Current behavior:

ProfileParameters Using Deprecated TimeNoMod & TimeLayout

{
	"response": [
		{
			"lastUpdated": "2023-08-22 09:51:03+05",
			"profile": "test2",
			"parameter": 1070
		},
		{
			"lastUpdated": "2023-08-22 10:33:23+05",
			"profile": "test2",
			"parameter": 1071
		}
        ]
}

Expected behavior:

ProfileParameters to Use RFC3339 Format

{
	"response": [
		{
			"lastUpdated": "2023-08-22T09:51:03.556698+05:30",
			"profile": "test2",
			"parameter": 1070
		},
		{
			"lastUpdated": "2023-08-22T10:33:23.249862+05:30",
			"profile": "test2",
			"parameter": 1071
		}
       ]
}

Steps to reproduce:

Make the above api call to 4.1 version

@jagan-parthiban jagan-parthiban added Traffic Ops related to Traffic Ops low impact affects only a small portion of a CDN, and cannot itself break one tech debt rework due to choosing easy/limited solution labels Aug 22, 2023
jagan-parthiban added a commit to jagan-parthiban/trafficcontrol that referenced this issue Aug 22, 2023
jagan-parthiban added a commit to jagan-parthiban/trafficcontrol that referenced this issue Aug 22, 2023
rimashah25 pushed a commit that referenced this issue Aug 24, 2023
…mat (#7738)

* RFC3339 Changes for ProfileParameters. Fixes #7737

* CHANGELOG.md entry added

* Fixed PR review comments

* Fixed PR review comments

* Remove strconv function and use APIInfo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low impact affects only a small portion of a CDN, and cannot itself break one tech debt rework due to choosing easy/limited solution Traffic Ops related to Traffic Ops
Projects
None yet
1 participant