Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v1.1.6 rollup #218

Merged
merged 35 commits into from
Oct 20, 2019
Merged

v1.1.6 rollup #218

merged 35 commits into from
Oct 20, 2019

Conversation

rhinof
Copy link
Contributor

@rhinof rhinof commented Oct 20, 2019

No description provided.

Guy Baron and others added 30 commits September 6, 2019 10:49
* added generic handler metrics with message type as the label

* add handler name label to the metrics

* adding new metrics to the read me
…160)

* set the correct Type and Content-Type headers on out going messages

* refactoring
…a the RPC interface (#163)


* fixing ReplyToInitiator not working  when initiator sends a message via the RPC interface
* better wording for documentation
* setting the target saga id on the saga correlation id field

* added emperror support
* fixed logging issues and added documentation

logging via the invocation interface was broken and did not add
contextual data related to the invocation due to a bug in the way the Glogged structure is currently implemented.

Also added documentation on how logging should be done within a
handler including adding context to returned errors so that data gets logged

* added missing documentation file
* fixed emperror url format

* added serialization documentation
… processing (#181)

* fixed emperror url format

* added serialization documentation

* added documentation for message semantics, retries and transactions
* fixed emperror url format

* added serialization documentation

* added documentation for message semantics, retries and transactions

* fixing tx documentation page
* fixing the way that the target service was resolved

closes #195

#195

* Fixing the value of the StartedBy field when creating a new saga
instance

closes #194
#194

* rolling back checking if replying to an event to maintain backward compatibility

* logging a Warn instead of rejecting the message when saga not found in store

closes #196
#196
Fixes issue #200
Also updated go.mod for newer versions of dependancies
* added metrics for transactional outbox

The follwoing metrics were added
outbox_total_records: reports the total amount of records currently in the outbox
outbox_pending_delivery: reports the total amount of records pending delivery currently in the outbox
outbox_pending_removal: reports the total amount of records that were sent and pending removal currently in the outbox

* reading status and count fields in the correct order from rows
…t or in saga store (#204)

* service name now gets added to log entries when a custom logger is set

#200

* fixing issue that the saga store was not adding the service to its log
entires

#206
* service name now gets added to log entries when a custom logger is set

#200

* fixing issue that the saga store was not adding the service to its log
entires

#206

* fixing minor issue with logging saga store initialization
* Added x-idempotency-key header and the ability for client code to set it

#106

* Set the value of BusMessage.ID as the default value of BusMessage.IdempotencyKey
Guy Baron and others added 5 commits October 16, 2019 10:23
* refactored returnDeadToQueue to allow correctly returning Events as well as Messages(commands)

* fixed lint issues

* code review fixes

* more review comments

* minor change to force coveralls to rebuild

* returnDeadToQueue - changed routing-key to always be that routing-key of the first death, not the latest death, also refactored naming and added comment for clarity

* moved getRoutingParamsFromDelivery to worker.go
* adding all logging context data to worker and saga log entries

* added logging with context when command or reply received for saga def but no saga correlation id found

* removing minor discrepancies and updating documentation
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.9%) to 76.278% when pulling ce8face on v1.x into eb32401 on master.

@rhinof rhinof merged commit 50aecfb into master Oct 20, 2019
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.

5 participants