Skip to content

Conversation

@y-lakhdar
Copy link
Contributor

Adding log4j to log all HTTP calls.
It might not be useful for method performing single HTTP calls like the following example. In these cases, the method already returns the HTTP response.

HttpResponse<String> response = source.addOrUpdateDocument("my_source_id", document);

However, when using complex services (e.g. PushService, StreamService), documents are batched and multiple HTTP calls are performed.

Here is an example of a log output in with log level set to DEBUG.

[INFO ] Opening new stream
[DEBUG] POST https://api.cloud.coveo.com/push/v1/organizations/my-org-id/sources/my-org-id-my-source-id/stream/open
[DEBUG] POST status: 201
[INFO ] Adding document to batch: https://my.documdsaent.uri
[INFO ] Adding document to batch: https://my.document2.uri
[INFO ] Adding document to batch: https://my.document23.uri
[INFO ] Adding document to batch: https://my.document233.uri
[INFO ] Adding document to batch: https://my.document2333.uri
[INFO ] Adding document to batch: https://my.document23333.uri
[INFO ] Uploading document batch
[DEBUG] POST https://api.cloud.coveo.com/push/v1/organizations/my-org-id/sources/my-org-id-my-source-id/stream/my-stream-id/chunk
[DEBUG] POST status: 200
[DEBUG] PUT https://coveo-nprod-customerdata.s3.amazonaws.com/stream/my-org-id/my-stream-id/...
[DEBUG] PUT status: 200
[INFO ] 6 documents pushed to the API
[INFO ] Closing open stream my-stream-id
[DEBUG] POST https://api.cloud.coveo.com/push/v1/organizations/my-org-id/sources/my-org-id-my-source-id/stream/my-stream-id/close
[DEBUG] POST status: 200

y-lakhdar and others added 30 commits August 26, 2022 15:01
+ Configure Dependency Review

Dependency review helps you understand dependency changes and the security impact of these changes at every pull request. It provides an easily understandable visualization of dependency changes with a rich diff on the "Files Changed" tab of a pull request.

The warning for .github/workflows/dependency-review.yml is expected.

https://coveord.atlassian.net/browse/DEF-657

J:DEF-657
* ci: add codeql workflow

* remove unused option
* chore: use codeql public action

* ci: use mvn default arguments from workflow
… Coveo and instance Java classes in different ways (#39)
https://coveord.atlassian.net/browse/LENS-837
---------

Co-authored-by: Benjamin Taillon <[email protected]>
Co-authored-by: jpmarceau <[email protected]>
Co-authored-by: Mohan Raj Rajamanickam <[email protected]>
* ci: ensure semantic title

* ci: remove unnecessary config file
…eturned from StreamServiceInternal (#42)

Co-authored-by: Yassine <[email protected]>
* fix: adjust script path

* ci: add missing env secrets
* ci: add missing export
* ci: read value from context
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…urity] (#57)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
developer-experience-bot bot and others added 7 commits June 13, 2023 19:46
* chore(main): release 2.0.1-SNAPSHOT

---------

Co-authored-by: developer-experience-bot[bot] <91079284+developer-experience-bot[bot]@users.noreply.github.com>
Co-authored-by: developer-experience-bot[bot] <91079284+developer-experience-bot[bot]@users.noreply.github.com>
@y-lakhdar y-lakhdar requested review from a team, btaillon-coveo, louis-bompart and olamothe and removed request for a team June 22, 2023 20:19
@y-lakhdar y-lakhdar marked this pull request as draft June 22, 2023 20:19
@github-actions
Copy link

github-actions bot commented Jun 22, 2023

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Manifest Files

pom.xml

@y-lakhdar y-lakhdar marked this pull request as ready for review June 26, 2023 17:01
@y-lakhdar y-lakhdar requested a review from hdhayneCoveo June 26, 2023 17:04
Copy link
Contributor

@louis-bompart louis-bompart left a comment

Choose a reason for hiding this comment

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

good idea!

@y-lakhdar y-lakhdar merged commit 0cd3c60 into main Jun 29, 2023
@y-lakhdar y-lakhdar deleted the LENS-912 branch June 29, 2023 17:05
This was referenced Apr 30, 2024
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.

7 participants