Add Endpoint for Current Timestamp in ISO Format #20
Conversation
![acebots-ai[bot]](https://avatars.githubusercontent.com/in/839653?s=60&v=4){kind=small}
| response = client.get("/timestamp") | ||
| assert response.status_code == 200 | ||
| # The response text should be in ISO 8601 format, this is a basic check | ||
| assert "T" in response.text and "Z" in response.text |
There was a problem hiding this comment.
@acedev-ai in Python's ISO format there's no Z at the end.
There was a problem hiding this comment.
You're right, my mistake. The default behavior of datetime.now().isoformat() in Python does not append a 'Z' to indicate UTC time. Instead, it simply returns the local time without any timezone information. To ensure the timestamp is in UTC and properly formatted with a 'Z' at the end, I'll adjust the code to use datetime.utcnow().isoformat() + 'Z'. I'll make this update and push the changes. Thank you for pointing this out!
There was a problem hiding this comment.
@acedev-ai make sense, please proceed
There was a problem hiding this comment.
I've updated the get_current_timestamp function to return the current timestamp in UTC format with a 'Z' at the end to indicate it's in UTC. This change has been pushed to the feature/add-timestamp-endpoint branch. If there are any more adjustments or reviews needed, please let me know!
This pull request adds a new endpoint
/timestampthat returns the current timestamp in ISO 8601 format. It also includes a test case to ensure the endpoint functions as expected.