Skip to content

Conversation

@areibman
Copy link
Contributor

No description provided.

Comment on lines 17 to +20
analyticsData: {
repoName: "agentops",
organization: "AgentOps-AI",
apiKey: "1234567890",
apiKey: "YrodtbYbdqWNKOraqTFuVWPQ5k6yqWHTevojAI_w0Zg",

Choose a reason for hiding this comment

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

API key is exposed in client-side JavaScript, creating a security vulnerability. Store sensitive credentials server-side or use environment variables.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
analyticsData: {
repoName: "agentops",
organization: "AgentOps-AI",
apiKey: "1234567890",
apiKey: "YrodtbYbdqWNKOraqTFuVWPQ5k6yqWHTevojAI_w0Zg",
analyticsData: {
repoName: "agentops",
organization: "AgentOps-AI",
apiKey: process.env.ANALYTICS_API_KEY

@areibman areibman merged commit 4105ed4 into main Feb 24, 2025
2 checks passed
@areibman areibman deleted the entelligence_docs branch February 24, 2025 23:55
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