fix: Agent panics with rustls CryptoProvider

[RambokDev]

Summary by CodeRabbit

• Improvements
  • Secure network connections are initialized with the AWS-LC cryptography provider during application startup, before logging is configured. This updates the startup sequence for applications that rely on Rustls-based connections.

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: Portabase/agent/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: e69f542a-fbd8-4342-a377-ef7636d4708f

📥 Commits

Reviewing files that changed from the base of the PR and between 82c9b0a and e1e6497.

⛔ Files ignored due to path filters (1)

• Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

• Cargo.toml
• src/main.rs

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

The change adds rustls 0.23 and attempts to install its AWS-LC crypto provider before logger initialization. The installation result is ignored.

Changes

Rustls startup

Layer / File(s)	Summary
Add and initialize rustls Cargo.toml, src/main.rs	Cargo.toml adds rustls 0.23. main attempts to install the AWS-LC crypto provider before logger initialization and ignores the result.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to e1e64

The startup change installs a Rustls provider before HTTP clients are created, and the reviewed change leaves no identified merge-blocking risk.

Architecture Summary

Architecture risk: 🔵 Low · up to e1e64

The change affects 2 systems.

Changed systems: src, Cargo.toml

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

• observed — src (api) was modified; 1 changed file maps to changed impact.
• observed — Cargo.toml (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

• observed — Modified behavior in Cargo.toml: Added the rustls dependency at version 0.23.
• observed — Modified behavior in src/main.rs: main attempts to install the AWS-LC rustls crypto provider before logger initialization, ignoring the installation result. The startup comment about removing locks was removed.

🚥 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 clearly describes the main change: preventing an Agent panic caused by the rustls CryptoProvider by installing the AWS-LC provider during startup.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 …
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.

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/main.rs	0.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!