Skip to content

fix(docs): correct broken AWS IAM Roles link in aws/s3 README#6259

Open
sanjibani wants to merge 2 commits into
crewAIInc:mainfrom
sanjibani:fix/broken-aws-iam-roles-link
Open

fix(docs): correct broken AWS IAM Roles link in aws/s3 README#6259
sanjibani wants to merge 2 commits into
crewAIInc:mainfrom
sanjibani:fix/broken-aws-iam-roles-link

Conversation

@sanjibani

@sanjibani sanjibani commented Jun 20, 2026

Copy link
Copy Markdown

Summary

  • Fix 404 link https://docs.aws.amazon.com/sdk-for-python/v1/developer-guide/iam-roles.html#creating-an-iam-role in lib/crewai-tools/src/crewai_tools/aws/s3/README.md.
  • AWS retired the v1 developer guide; canonical replacement is https://docs.aws.amazon.com/IAM/latest/UserGuide/id_roles.html (verified 200).

Test plan

  • New URL returns 200.
  • Old URL confirmed 404.

Summary by CodeRabbit

  • Documentation
    • Updated AWS connectivity documentation link to current IAM resources
    • Updated Tavily extractor tool documentation link to general API documentation

…tool

Tavily restructured their docs root; the canonical landing page is
`docs.tavily.com` which redirects to `/welcome`. Dropped the stale
`#extract` fragment.
`/sdk-for-python/v1/developer-guide/iam-roles.html` was retired when
AWS folded the v1 developer guide into the IAM User Guide. Linking to
the canonical `/IAM/latest/UserGuide/id_roles.html` page (verified
200).

@corridor-security corridor-security Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary: This PR updates documentation links for AWS IAM roles and Tavily API references, with no changes to executable code, authentication, authorization, data handling, or external integration behavior. No exploitable security vulnerabilities were identified.

Risk: Low risk. The changes are limited to README documentation URLs and do not introduce new attack surfaces or alter security boundaries.

@coderabbitai

coderabbitai Bot commented Jun 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: acad1049-cfb5-4b3e-9551-4b59c6bd791b

📥 Commits

Reviewing files that changed from the base of the PR and between 9db2d44 and 8357996.

📒 Files selected for processing (2)
  • lib/crewai-tools/src/crewai_tools/aws/s3/README.md
  • lib/crewai-tools/src/crewai_tools/tools/tavily_extractor_tool/README.md

📝 Walkthrough

Walkthrough

Two README documentation files have single-line URL updates: the AWS S3 README updates the IAM Roles link to the IAM/latest page, and the Tavily extractor README updates the response format documentation link from a Python SDK #extract anchor to the general Tavily docs homepage.

Changes

Documentation Link Fixes

Layer / File(s) Summary
Update documentation hyperlinks in S3 and Tavily READMEs
lib/crewai-tools/src/crewai_tools/aws/s3/README.md, lib/crewai-tools/src/crewai_tools/tools/tavily_extractor_tool/README.md
AWS IAM Roles link updated to the IAM/latest URL; Tavily extractor response format link changed from the Python SDK #extract anchor to the general https://docs.tavily.com homepage.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title specifically mentions the AWS IAM Roles link fix in aws/s3 README, which is the primary focus of the PR. However, the PR also includes a second documentation fix for the Tavily extractor tool that is not reflected in the title.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant