Skip to content

Make agent actions require POST#7416

Open
labkey-tchad wants to merge 7 commits intodevelopfrom
fb_chatCleanup
Open

Make agent actions require POST#7416
labkey-tchad wants to merge 7 commits intodevelopfrom
fb_chatCleanup

Conversation

@labkey-tchad
Copy link
Member

Rationale

Seems weird to have chat prompts on the request URL (also puts a limit on prompt size).
Also, some other little cleanup I did during the AI workshops.

Related Pull Requests

  • N/A

Changes

  • Make existing agent actions require CSRF
  • Clean up various IDE warnings
  • POST to ChatEndpointAction and QueryAgentAction

@labkey-matthewb
Copy link
Contributor

Nick has a PR too, should make sure this doesn't break query chat. #7415

@labkey-tchad
Copy link
Member Author

Nick has a PR too, should make sure this doesn't break query chat. #7415

queryAgent and chatEndpoint are both working for me with these changes.

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.

2 participants