feat: add support for Logs API#96
Merged
christina-de-martinez merged 1 commit intomainfrom Apr 8, 2026
Merged
Conversation
There was a problem hiding this comment.
1 issue found across 5 files
Confidence score: 3/5
- There is a concrete security/compliance risk in
src/Resend/IResend.cs: newly added Resend Logs SDK methods are missing the explicit API-key permission verification reminder required by the “API Key Permission Check SDK Methods” rule. - Because this issue is high severity (8/10) with high confidence (10/10), the merge carries some user-impact risk if consumers call these methods without clear permission guidance.
- Pay close attention to
src/Resend/IResend.cs- ensure the new SDK methods include the required API-key permission verification reminder before merging.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="src/Resend/IResend.cs">
<violation number="1" location="src/Resend/IResend.cs:977">
P1: Custom agent: **API Key Permission Check SDK Methods**
This adds new Resend Logs SDK methods without an explicit API-key permission verification reminder required by the “API Key Permission Check SDK Methods” rule; confirm production keys can access Logs endpoints before release.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
drish
approved these changes
Apr 8, 2026
This was referenced Apr 10, 2026
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by cubic
Adds support for the Logs API to list and retrieve API request logs with pagination. Single-log retrieval includes request and response bodies; tests and local endpoints are included.
Logmodel with metadata plus optionalrequest_bodyandresponse_body.IResendmethods:LogListAsync(PaginatedQuery?)andLogRetrieveAsync(Guid).GET /logs(supportslimit,before,after) andGET /logs/{id}.tools/Resend.ApiServerto simulate the Logs API.Written for commit e4db939. Summary will update on new commits.