Skip to content

Commit

Permalink
fix: Correct API endpoint and adjust localhost configuration (Resolves
Browse files Browse the repository at this point in the history
…#346) (#349)

## Title

fix: Correct API endpoint and adjust localhost configuration (Resolves
#346)

##  Description:

This pull request addresses the following issues related to the API call
and local development setup:
• API Endpoint Fix: Updated the endpoint path from
foyle.logs.SessionsService/GetSession to api/GetSession for proper
routing.
• jq Syntax Correction: Added a period (.) after jq to ensure JSON
output is parsed and displayed correctly.
• Protocol Adjustment: Replaced https:// with http:// for localhost to
avoid unnecessary SSL complications during local testing.

### Before
```
CONTEXTID=01J7S3QZMS5F742JFPWZDCTVRG
curl -s -X POST https://localhost:8877/foyle.logs.SessionsService/GetSession -H "Content-Type: application/json" -d "{\"contextId\": \"${CONTEXTID}\"}" | jq.
```

### After
```
CONTEXTID=01J7S3QZMS5F742JFPWZDCTVRG
curl -s -X POST http://localhost:8877/api/GetSession -H "Content-Type: application/json" -d "{\"contextId\": \"${CONTEXTID}\"}" | jq .
```
Closes Issue: #346

---------

Co-authored-by: Indy S <[email protected]>
  • Loading branch information
IndySeh and Indy S authored Dec 3, 2024
1 parent 2c1180c commit 6509cc0
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ You can use the session ID to track whether learning occured.

```bash
CONTEXTID=01J7S3QZMS5F742JFPWZDCTVRG
curl -s -X POST https://localhost:8877/foyle.logs.SessionsService/GetSession -H "Content-Type: application/json" -d "{\"contextId\": \"${CONTEXTID}\"}" | jq.
curl -s -X POST http://localhost:8877/api/foyle.logs.SessionsService/GetSession -H "Content-Type: application/json" -d "{\"contextId\": \"${CONTEXTID}\"}" | jq .
```

* If this returns not found then no log was created for this sessions and there is a problem with Log Processing
Expand Down

0 comments on commit 6509cc0

Please sign in to comment.