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

[INTPROD-9204] Add Auth-Test Call #107

Merged
merged 3 commits into from
Apr 5, 2024
Merged

[INTPROD-9204] Add Auth-Test Call #107

merged 3 commits into from
Apr 5, 2024

Conversation

colinsl
Copy link
Contributor

@colinsl colinsl commented Apr 4, 2024

This pull request includes a significant change to the omnibot/services/slack/__init__.py file. A new function get_auth(bot) has been added. This function retrieves the authentication information for the bot. If the authentication information is not found in the Redis client, it makes an API call to auth.test. If the API call is successful, the authentication information is stored in the Redis client for future use. If the API call fails, a warning is logged and an empty dictionary is returned.

  • omnibot/services/slack/__init__.py: Added a new function get_auth(bot). This function retrieves the bot's authentication information from the Redis client, or makes an API call to auth.test if the information is not found. The result of the API call is then stored in the Redis client. If the API call fails, a warning is logged and an empty dictionary is returned.

@colinsl
Copy link
Contributor Author

colinsl commented Apr 4, 2024

PTAL @ryand0 @toverturf-lyft @david-lyft

remove space
david-lyft
david-lyft previously approved these changes Apr 5, 2024
follow upstream
@colinsl colinsl merged commit 9567a20 into master Apr 5, 2024
5 checks passed
@colinsl colinsl deleted the colinsl/auth-test branch April 5, 2024 16:15
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.

2 participants