Skip to content

fix: updated taiga auth#13

Merged
irfanuddinahmad merged 1 commit intoproductionfrom
fix/taiga-auth
Mar 3, 2026
Merged

fix: updated taiga auth#13
irfanuddinahmad merged 1 commit intoproductionfrom
fix/taiga-auth

Conversation

@irfanuddinahmad
Copy link

This pull request introduces improvements to Taiga connection authentication and enhances the DeveloperTelemetry Grafana dashboard to support dynamic connection IDs. It also includes minor UI enhancements for Taiga connection configuration.

Taiga Connection Authentication Improvements

  • Added logic to TaigaConn in connection.go to support authentication via username and password by exchanging credentials for a token if a token is not provided. This includes a new fetchToken method that performs a POST request to Taiga's /auth endpoint and handles both v5 and v6 token formats.

  • Updated imports in connection.go to support new authentication logic, including adding bytes, encoding/json, fmt, io, and strings.

Grafana Dashboard Enhancements

  • Updated all SQL queries in DeveloperTelemetry.json to use a dynamic ${connection_id} variable instead of hardcoded values, allowing the dashboard to work with multiple connections. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13]
  • Minor formatting updates to legend and reducer configurations for improved readability and consistency. [1] [2] [3] [4] [5] [6] [7]

Taiga Connection UI Improvements

  • Updated the endpoint placeholder in the Taiga connection UI to a more realistic example.
  • Improved the username field placeholder to clarify that either username or email can be used.
  • Changed the password field placeholder to show masked text during updates, enhancing UX.
  • Removed unused useState import from the Taiga connection UI component.

@irfanuddinahmad irfanuddinahmad merged commit e2114d6 into production Mar 3, 2026
5 checks passed
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.

1 participant