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.

Sign up for GitHub

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

Fix #1418 Expose AwsCredentialsProvider to AmazonS3InstallationService #1419

Merged

seratch merged 1 commit into slackapi:main from seratch:issue-1418

Jan 15, 2025

+57 −8

Merged

Fix #1418 Expose AwsCredentialsProvider to AmazonS3InstallationService #1419

Fix #1418 Expose AwsCredentialsProvider to AmazonS3InstallationService

Codecov / codecov/patch succeeded Jan 14, 2025 in 1s

57.14% of diff hit (target 30.00%)

View this Pull Request on Codecov

57.14% of diff hit (target 30.00%)

Annotations

Check warning on line 44 in bolt/src/main/java/com/slack/api/bolt/service/builtin/AmazonS3InstallationService.java

codecov / codecov/patch

bolt/src/main/java/com/slack/api/bolt/service/builtin/AmazonS3InstallationService.java#L43-L44

Added lines #L43 - L44 were not covered by tests

Check warning on line 332 in bolt/src/main/java/com/slack/api/bolt/service/builtin/AmazonS3InstallationService.java

codecov / codecov/patch

bolt/src/main/java/com/slack/api/bolt/service/builtin/AmazonS3InstallationService.java#L332

Added line #L332 was not covered by tests

Check warning on line 130 in bolt/src/main/java/com/slack/api/bolt/service/builtin/AmazonS3OAuthStateService.java

codecov / codecov/patch

bolt/src/main/java/com/slack/api/bolt/service/builtin/AmazonS3OAuthStateService.java#L126-L130

Added lines #L126 - L130 were not covered by tests

View more details on Codecov