Skip to content
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

feat: update memo detail page #1682

Merged
merged 2 commits into from
May 20, 2023
Merged

feat: update memo detail page #1682

merged 2 commits into from
May 20, 2023

Conversation

boojack
Copy link
Collaborator

@boojack boojack commented May 20, 2023

No description provided.

@github-actions
Copy link

github-actions bot commented May 20, 2023

Uffizzi Ephemeral Environment deployment-25951

☁️ https://app.uffizzi.com/github.com/usememos/memos/pull/1682

📄 View Application Logs etc.

What is Uffizzi? Learn more!

@boojack boojack merged commit b405710 into main May 20, 2023
9 checks passed
@github-actions
Copy link

Uffizzi Preview deployment-25951 was deleted.

if c.QueryParam("thumbnail") == "1" && common.HasPrefixes(resource.Type, "image/png", "image/jpeg") {
ext := filepath.Ext(filename)
thumbnailPath := path.Join(s.Profile.Data, thumbnailImagePath, resource.PublicID+ext)
if _, err := os.Stat(thumbnailPath); err != nil {

Check failure

Code scanning / CodeQL

Uncontrolled data used in path expression High

This path depends on a
user-provided value
.
}
}

src, err := os.Open(thumbnailPath)

Check failure

Code scanning / CodeQL

Uncontrolled data used in path expression High

This path depends on a
user-provided value
.
@boojack boojack deleted the feat/update-memo-detail-page branch May 21, 2023 13:01
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.

None yet

1 participant