Skip to content

Projects integration#2057

Draft
emranemran wants to merge 7 commits intomasterfrom
emran/projects-integration
Draft

Projects integration#2057
emranemran wants to merge 7 commits intomasterfrom
emran/projects-integration

Conversation

@emranemran
Copy link
Copy Markdown
Contributor

WIP integration branch

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
livepeer-studio ❌ Failed (Inspect) Feb 27, 2024 0:54am

Comment thread packages/api/src/controllers/project.ts Fixed
Comment thread packages/api/src/controllers/project.ts Fixed

const app = Router();

app.get("/", authorizer({}), async (req, res) => {

Check failure

Code scanning / CodeQL

Missing rate limiting

This route handler performs [authorization](1), but is not rate-limited.
res.json({});
});

app.post("/", authorizer({}), async (req, res) => {

Check failure

Code scanning / CodeQL

Missing rate limiting

This route handler performs [authorization](1), but is not rate-limited.
return project;
}

app.get("/:projectId", authorizer({}), async (req, res) => {

Check failure

Code scanning / CodeQL

Missing rate limiting

This route handler performs [authorization](1), but is not rate-limited.
res.json(project);
});

app.post("/", authorizer({}), async (req, res) => {

Check failure

Code scanning / CodeQL

Missing rate limiting

This route handler performs [authorization](1), but is not rate-limited.
userId: `project.data->>'userId'`,
} as const;

app.get("/", authorizer({}), async (req, res) => {

Check failure

Code scanning / CodeQL

Missing rate limiting

This route handler performs [authorization](1), but is not rate-limited.
@emranemran emranemran changed the title some basic logic around projects in stream.ts Projects integration Feb 21, 2024
@0xcadams
Copy link
Copy Markdown
Contributor

Can we close this now that we landed projects?

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.

4 participants