Skip to content

fix: remove debug logs and update stale DNS comments in agent-secret - #317

Merged
Haven Xia (HavenXia) merged 7 commits into
agent-substrate:mainfrom
mesutoezdil:fix/agent-secret-debug-cleanup
Jul 23, 2026
Merged

fix: remove debug logs and update stale DNS comments in agent-secret#317
Haven Xia (HavenXia) merged 7 commits into
agent-substrate:mainfrom
mesutoezdil:fix/agent-secret-debug-cleanup

Conversation

@mesutoezdil

@mesutoezdil Mesut Oezdil (mesutoezdil) commented Jun 26, 2026

Copy link
Copy Markdown
Contributor
  • Remove two DEBUG log lines left in production handler
  • Remove dead code block (actorID is always non-empty after the unknown fallback)

@mesutoezdil
Mesut Oezdil (mesutoezdil) force-pushed the fix/agent-secret-debug-cleanup branch 2 times, most recently from 37baa76 to 759d385 Compare July 14, 2026 19:00
@mesutoezdil
Mesut Oezdil (mesutoezdil) force-pushed the fix/agent-secret-debug-cleanup branch from 759d385 to b0a00ff Compare July 16, 2026 06:43
…g-cleanup

# Conflicts:
#	demos/agent-secret/main.go
@mesutoezdil

Copy link
Copy Markdown
Contributor Author

Haven Xia (@HavenXia) would you mind taking a look when you get a chance, since you touched this file recently?

@HavenXia Haven Xia (HavenXia) self-assigned this Jul 17, 2026
@HavenXia
Haven Xia (HavenXia) self-requested a review July 17, 2026 17:46

@HavenXia Haven Xia (HavenXia) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: The 2nd point in description says "Update stale substrate.k8s.io references to substrate.ate.dev in comments" but seems it has been cleared, please remove.

@mesutoezdil

Copy link
Copy Markdown
Contributor Author

Thanks for the review!

@HavenXia
Haven Xia (HavenXia) merged commit 5e059fd into agent-substrate:main Jul 23, 2026
11 checks passed
@mesutoezdil
Mesut Oezdil (mesutoezdil) deleted the fix/agent-secret-debug-cleanup branch July 25, 2026 10:22
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