Skip to content

Conversation

@shivam-kumar-ts
Copy link

@shivam-kumar-ts shivam-kumar-ts commented Sep 18, 2025

  • Update README.md to change token server command from 'start-token-server' to 'token-server'
  • format code for consistency
  • Add Accept-Language header to bearer authentication config in ThoughtSpot client

@codesandbox
Copy link

codesandbox bot commented Sep 18, 2025

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@vercel
Copy link

vercel bot commented Sep 18, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
mcp-chat-client Ready Ready Preview Comment Sep 22, 2025 3:30pm

@mnk-blr
Copy link
Contributor

mnk-blr commented Sep 18, 2025

Snyk checks have failed. 60 issues have been found so far.

Icon Severity Issues
Critical 1
High 17
Medium 42
Low 0

⚠️ security/snyk check encountered an error. (View Details)

⚠️ license/snyk check encountered an error. (View Details)

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

getCachedAuthToken,
{
additionalHeaders: {
"Accept-Language": "ThoughtSpot-SDK",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Accept-Language cannot be ThoughtSpot SDK. It is wrong. It has to be a language like en-US

Copy link
Author

Choose a reason for hiding this comment

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

Done the required 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.

4 participants