Skip to content

Conversation

@yonahforst
Copy link
Contributor

No description provided.

app.get('/:providerName/authorized', middlewares.hasSessionAndProvider, middlewares.gentleVerifyToken, controllers.authorized)
app.get('/:providerName/list/:id?', middlewares.hasSessionAndProvider, middlewares.verifyToken, controllers.list)
app.post('/:providerName/get/:id', middlewares.hasSessionAndProvider, middlewares.verifyToken, controllers.get)
app.post('/:providerName/get/:id', middlewares.hasSessionAndProvider, middlewares.gentleVerifyToken, controllers.get)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you mind sharing what is the reason for this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh crap 💩, that was sloppy of me.

I was trying to solve #1303 and commited that by accident. I'll remove it now :)

This reverts commit eb6098e.
@arturi
Copy link
Contributor

arturi commented Mar 18, 2019

@ifedapoolarewaju is this good after the fix?

@ifedapoolarewaju
Copy link
Contributor

it looks good to me, yes

@ifedapoolarewaju ifedapoolarewaju merged commit 29f0f25 into transloadit:master Mar 18, 2019
HeavenFox pushed a commit to docsend/uppy that referenced this pull request Jun 27, 2023
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.

3 participants