Skip to content

Commit 7d06fb7

Browse files
DeanChensjcopybara-github
authored andcommitted
chore: Move create_session log to where the session is actually created
PiperOrigin-RevId: 789094066
1 parent d5dcef2 commit 7d06fb7

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/google/adk/cli/adk_web_server.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -423,14 +423,14 @@ async def create_session_with_id(
423423
)
424424
is not None
425425
):
426-
logger.warning("Session already exists: %s", session_id)
427426
raise HTTPException(
428427
status_code=400, detail=f"Session already exists: {session_id}"
429428
)
430-
logger.info("New session created: %s", session_id)
431-
return await self.session_service.create_session(
429+
session = await self.session_service.create_session(
432430
app_name=app_name, user_id=user_id, state=state, session_id=session_id
433431
)
432+
logger.info("New session created: %s", session_id)
433+
return session
434434

435435
@app.post(
436436
"/apps/{app_name}/users/{user_id}/sessions",
@@ -442,7 +442,6 @@ async def create_session(
442442
state: Optional[dict[str, Any]] = None,
443443
events: Optional[list[Event]] = None,
444444
) -> Session:
445-
logger.info("New session created")
446445
session = await self.session_service.create_session(
447446
app_name=app_name, user_id=user_id, state=state
448447
)
@@ -451,6 +450,7 @@ async def create_session(
451450
for event in events:
452451
await self.session_service.append_event(session=session, event=event)
453452

453+
logger.info("New session created")
454454
return session
455455

456456
@app.post(

0 commit comments

Comments
 (0)