-
Notifications
You must be signed in to change notification settings - Fork 3.1k
v0.5.36: hitl improvements, opengraph, slack fixes, one-click unsubscribe, auth checks, new db indexes #2474
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
Conversation
* fixed slack output * updated jira * removed comment * change team uuid
Co-authored-by: aadamgough <aadamgough@users.noreply.github.com>
…2464) * fixed the human in the loop url resolution: * greptilecomments * greptilecomments --------- Co-authored-by: Pbonmars-20031006@users.noreply.github.com
…ting opengraph image for all other pages (#2466) * feat(og): add opengraph images for templates, blogs, and updated existing opengraph image for all other pages * added to workspace templates page as well * ack PR comments
…ntations (#2469) * fix(slack): respect message limit, remove duplicate canonical representations * removed comment * updated docs script --------- Co-authored-by: aadamgough <adam@sim.ai>
* fix(unsubscribe): add one-click unsubscribe * ack Pr comments
…postgres query validation, csp improvements (#2472)
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile SummaryThis PR consolidates multiple security, performance, and UX improvements across 9 merged PRs. The changes significantly strengthen the application's security posture through comprehensive authentication checks, SQL/NoSQL injection prevention, and CSP hardening while adding useful features like one-click unsubscribe and OpenGraph images. Key Changes
Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant Client
participant API
participant Auth
participant Perm
participant DB
participant Valid
participant Email
Note over Client,Email: Authentication and Authorization Flow
Client->>API: API Request
API->>Auth: Check authentication
Auth-->>API: User ID
API->>Perm: Verify permissions
Perm->>DB: Query memberships
DB-->>Perm: Membership data
Perm-->>API: Permission result
alt Unauthorized
API-->>Client: 403 Error
else Authorized
API->>DB: Perform operation
DB-->>API: Data
API-->>Client: Success
end
Note over Client,Email: Query Validation
Client->>API: Database query request
API->>Valid: Validate query
Valid->>Valid: Check patterns
alt Dangerous pattern
Valid-->>API: Error
API-->>Client: 400 Error
else Safe
Valid-->>API: Approved
API->>DB: Execute
DB-->>API: Data
API-->>Client: Success
end
Note over Client,Email: Unsubscribe Flow
Client->>API: Unsubscribe request
API->>API: Verify token
alt Valid
API->>DB: Update preferences
DB-->>API: Updated
API-->>Client: Success
else Invalid
API-->>Client: 401 Error
end
Note over Client,Email: Email with Unsubscribe
Email->>Email: Prepare message
Email->>Email: Check preferences
alt Unsubscribed
Email-->>Email: Skip
else Subscribed
Email->>Email: Add headers
Email->>Email: Send message
end
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
77 files reviewed, 1 comment
* fix(sanitization): added more input sanitization to tool routes * ack PR comments
Uh oh!
There was an error while loading. Please reload this page.