vodorod/Dorod-Sky
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

add some logs for debug_session endpoint (#3056)

Browse Source
This commit is contained in:
Jonathan Dobson
2025-07-29 21:04:30 -04:00
committed by GitHub
parent 083d54f018
commit 33a801429b
1 changed files with 45 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
skyvern/forge/sdk/routes/agent_protocol.py
View File

@@ -2048,8 +2048,26 @@ async def get_or_create_debug_session_by_user_and_workflow_permanent_id(
workflow_permanent_id=workflow_permanent_id, workflow_permanent_id=workflow_permanent_id,
) )
LOG.info(
"Existing debug session not found, created a new one",
debug_session_id=debug_session.debug_session_id,
browser_session_id=new_browser_session.persistent_browser_session_id,
organization_id=current_org.organization_id,
user_id=current_user_id, # not sure how we feel about logging this
workflow_permanent_id=workflow_permanent_id,
)
return debug_session return debug_session
LOG.info(
"Existing debug session found",
debug_session_id=debug_session.debug_session_id,
browser_session_id=debug_session.browser_session_id,
organization_id=current_org.organization_id,
user_id=current_user_id, # not sure how we feel about logging this
workflow_permanent_id=workflow_permanent_id,
)
browser_session = await app.DATABASE.get_persistent_browser_session( browser_session = await app.DATABASE.get_persistent_browser_session(
debug_session.browser_session_id, current_org.organization_id debug_session.browser_session_id, current_org.organization_id
) )
@@ -2097,7 +2115,24 @@ async def get_or_create_debug_session_by_user_and_workflow_permanent_id(
return debug_session return debug_session
else: else:
LOG.info( LOG.info(
"pbs for debug session has expired", "Browser session for debug session has expired",
debug_session_id=debug_session.debug_session_id,
organization_id=current_org.organization_id,
workflow_permanent_id=workflow_permanent_id,
user_id=current_user_id,
)
else:
if browser_session:
LOG.info(
"Browser session for debug session has expired, creating a new one",
debug_session_id=debug_session.debug_session_id,
organization_id=current_org.organization_id,
workflow_permanent_id=workflow_permanent_id,
user_id=current_user_id,
)
else:
LOG.info(
"Browser session did not exist for debug session, creating a new one",
debug_session_id=debug_session.debug_session_id, debug_session_id=debug_session.debug_session_id,
organization_id=current_org.organization_id, organization_id=current_org.organization_id,
workflow_permanent_id=workflow_permanent_id, workflow_permanent_id=workflow_permanent_id,
@@ -2114,4 +2149,13 @@ async def get_or_create_debug_session_by_user_and_workflow_permanent_id(
browser_session_id=browser_session.persistent_browser_session_id, browser_session_id=browser_session.persistent_browser_session_id,
) )
LOG.info(
"Updated debug session with new browser session",
debug_session_id=debug_session.debug_session_id,
browser_session_id=browser_session.persistent_browser_session_id,
organization_id=current_org.organization_id,
workflow_permanent_id=workflow_permanent_id,
user_id=current_user_id,
)
return debug_session return debug_session