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

API actions (WIP) #2

Closed
wants to merge 1 commit into from
Closed

API actions (WIP) #2

wants to merge 1 commit into from

Conversation

murrant
Copy link

@murrant murrant commented Feb 17, 2019

No idea if this is acceptable or working. I just threw some rough code together for an idea I had. (Disclaimer, I'm not a JS developer)

My use case is to post a message to a chat via API.
No idea how to:

  • test this code
  • obscure sensitive data like API tokens

If this is acceptable let me know and I'll keep working.

@dessant
Copy link
Owner

dessant commented Feb 17, 2019

It is a cool idea, but this would allow people to make arbitrary requests using the hosted instance of the app. As a general guideline, only actions which do not call third-party sevices are in scope for now.

@murrant
Copy link
Author

murrant commented Feb 18, 2019

Indeed, so I guess specific APIs would have to be implemented for safety. That doesn't sound like a very fun path maintenance wise.

Close this down for now? or try to come up with a way to support multiple chat api "plugins"?

There also might be another way to achieve what I am looking for. I'll do some research.

@dessant
Copy link
Owner

dessant commented Feb 18, 2019

Yeah, these features might be better suited for a dedicated bot for posting to different chat platforms.

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

2 participants