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

Huijie - fix recalculation for tangible hours #1093

Merged

Conversation

metaphor987
Copy link
Contributor

Description

the API for recalculating tangible hours for all users in PR #1004 got errors due to Azure's time limit for idle TCP connections. This PR sends "keep-alive" messages every 150 seconds to the connection active.

Related PRS (if any):

This backend PR is a fix for #1004

How to test:

  1. check into current branch
  2. do npm install, npm run build and npm start to run this PR locally
  3. Clear site data/cache
  4. log as any user
  5. (before merging) use Postman to send POST request to http://localhost:4500/api/TimeEntry/recalculateHoursAllUsers/tangible
  6. (after merging) use Postman to send POST request to https://hgn-rest-dev.azurewebsites.net/api/TimeEntry/recalculateHoursAllUsers/tangible for Dev or https://hgn-rest-beta.azurewebsites.net/api/TimeEntry/recalculateHoursAllUsers/tangible for Prod and check the response messages

@ArtemisNyx3
Copy link

ArtemisNyx3 commented Sep 12, 2024

Hi @metaphor987
I am getting an unauthorized request error if I try to run it with an admin account. Could you please help me if I had the wrong Postman setup.

image

@one-community one-community merged commit b5d5536 into development Sep 12, 2024
3 checks passed
@metaphor987
Copy link
Contributor Author

Hi @ArtemisNyx3,
Thank you for reviewing! This PR has already been merged as a test for a quick fix. Sorry that I didn't make it clear in the comments earlier.
For the Postman setup, I put the Authorization key in the following place and it worked:
Screenshot 2024-09-14 at 11 29 08 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants