Skip to content

Conversation

@urnotsam
Copy link
Contributor

@urnotsam urnotsam commented Jul 31, 2025

PR Type

Bug fix


Description

  • Fix log API endpoint in buildLogAPIUrl method

Changes walkthrough 📝

Relevant files
Bug fix
Collector.ts
Update log API endpoint in Collector                                         

src/external/Collector.ts

  • Changed log API endpoint from /api/v2/logs to /api/log
  • Ensures correct URL is used for log requests
  • +1/-1     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @github-actions
    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🏅 Score: 98
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    API Endpoint Change

    The log API endpoint was changed from /api/v2/logs to /api/log. Ensure that this change is intentional and compatible with all consumers and backend services.

    const apiUrl = `${baseDomain}/api/log`


    buildLogAPIUrl(request: any, baseDomain = CONFIG.explorerUrl): string {
    const apiUrl = `${baseDomain}/api/v2/logs`
    const apiUrl = `${baseDomain}/api/log`

    Choose a reason for hiding this comment

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

    Suggestion: Ensure that the new endpoint /api/log matches the expected API route on the backend. If the backend expects /api/logs or a different path, this change could break logging functionality. [possible issue, importance: 7]

    Suggested change
    const apiUrl = `${baseDomain}/api/log`
    const apiUrl = `${baseDomain}/api/logs`

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    3 participants