Wojciech Kapcia (Tigase) opened 6 months ago
-
-
-
-
Name Previous Value Current Value Assignee
wojtek
andrzej.wojcik
-
Name Previous Value Current Value Version
empty 2.5.0
-
Andrzej Wójcik (Tigase) changed state to 'In Progress' 1 week ago
Previous Value Current Value Open
In Progress
-
I've reimplemented S3 support (replaced usage of AWS library), moved S3 store implementation to HTTP API component, and got rid of
tigase-extras-awsmodule. I've also updated dependencies in distribution module (please review this change).I've noted that
tigase-extras-awswas also used to provide support forcloudwatch, so please review if it's removal is correct.The new implementation was tested manually and I've added test case that uses TestContainers (if docker is available) to validate implementation behavior.
-
Name Previous Value Current Value State
In Progress
In QA
Assignee
andrzej.wojcik
wojtek
-
I've reimplemented S3 support (replaced usage of AWS library), moved S3 store implementation to HTTP API component, and got rid of
tigase-extras-awsmodule. I've also updated dependencies in distribution module (please review this change).Looks OK.
I removed the dependencies from dist package.
Should we keep
log.infoentries (in servlet/deployment info)? Or those are remnants from other issue/debugging? They could be helpful but maybe withlog.fineor somesuch?I've noted that
tigase-extras-awswas also used to provide support forcloudwatch, so please review if it's removal is correct.I think it's OK. I'd guess usage was low and even we dropped it because it wasn't efficient from cost point of view (and it's possible to configure logback to achieve the same in more performant way)
The new implementation was tested manually and I've added test case that uses TestContainers (if docker is available) to validate implementation behavior.
Great call for using testcontainers!
-
Previous Value Current Value In QA
Closed
-
Should we keep log.info entries (in servlet/deployment info)? Or those are remnants from other issue/debugging? They could be helpful but maybe with log.fine or somesuch?
We can use
log.fine, I recall I encounteredlog.infoso I've left them (or maybe merge restored them?).
| Type |
Task
|
| Priority |
Normal
|
| Assignee | |
| Version |
2.5.0
|
| Sprints |
n/a
|
| Customer |
n/a
|
-
tigase-server-8.5.0 Open
AWS (S3) SDK is huge cow, we should switch to smaller alternative. https://github.com/davidmoten/aws-lightweight-client-java seems like the best candidate (single binary without dependencies, <100KB).