heplify-server icon indicating copy to clipboard operation
heplify-server copied to clipboard

parse Via and add via_branch to custom header

Open silver0511 opened this issue 3 years ago • 20 comments
trafficstars

roadmap

Please put the via branch to the custom headers. It's not "must have" value

hi i have moved via branch to custome header, please check and review, thanks

hi i have uploaded two videos to show the transaction features. the first is a call with multiple session. the second is a call with complex sips. (idea from: https://thanhloi2603.wordpress.com/2017/04/22/understanding-sip-transaction-and-dialog/ )

https://user-images.githubusercontent.com/3271554/156127422-a94935a6-1bc9-4c99-9f62-d7960c9853b2.mp4

https://user-images.githubusercontent.com/3271554/156127453-da4c8135-ae0b-4e72-bbb3-64cdcaac3d96.mp4

feature_transaction pr: homer-app: https://github.com/sipcapture/homer-app/pull/474 homer-ui: https://github.com/sipcapture/homer-ui/pull/584

silver0511 avatar Feb 07 '22 11:02 silver0511

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Feb 07 '22 11:02 CLAassistant

Please put the via branch to the custom headers. It's not "must have" value

adubovikov avatar Feb 21 '22 08:02 adubovikov

Please put the via branch to the custom headers. It's not "must have" value

@adubovikov hi, i have uploaded two videos to show the transaction features, thanks

silver0511 avatar Mar 01 '22 07:03 silver0511

@silver0511 did you make the PR for UI ?

adubovikov avatar Mar 01 '22 10:03 adubovikov

@silver0511 did you make the PR for UI ?

@adubovikov i will make the pr for ui, homer-app together as soon as possible, thanks

silver0511 avatar Mar 03 '22 01:03 silver0511

@silver0511 did you make the PR for UI ?

@adubovikov hi, i have maked pr for homer-app(https://github.com/sipcapture/homer-app/pull/474) and homer-ui(https://github.com/sipcapture/homer-ui/pull/584), thanks

silver0511 avatar Mar 08 '22 09:03 silver0511

@silver0511 did you make the PR for UI ?

@adubovikov hi, i have maked pr for homer-app(sipcapture/homer-app#474) and homer-ui(sipcapture/homer-ui#584), thanks

@adubovikov hello

silver0511 avatar Mar 28 '22 09:03 silver0511

@silver0511 thanks for your PR, you know we appreciate it but unfortunately its a difficult time. we're going to review and adapt your changes as soon as viable for our team

lmangani avatar Mar 28 '22 09:03 lmangani

@lmangani Thanks for your recognition, i will always cooperate with you to adjust, fighting!!! :)

silver0511 avatar Mar 28 '22 10:03 silver0511

@silver0511 thanks for your PR, you know we appreciate it but unfortunately its a difficult time. we're going to review and adapt your changes as soon as viable for our team

@lmangani hi, is everything getting better now :)

silver0511 avatar Jun 07 '22 08:06 silver0511

@adubovikov hi adubovikov, need your help :)

silver0511 avatar Aug 04 '22 02:08 silver0511

@silver0511 apologies for this incubating so long! I don't want you to think it was not appreciated! We're now working on the next HOMER version and will make sure this change finds its final destination.

lmangani avatar Aug 09 '23 09:08 lmangani

@lmangani thanks for your reply, and i found new interesting feature in heplify and commited pull request, i really hope it‘s useful for you :)

silver0511 avatar Aug 31 '23 03:08 silver0511

@silver0511 the direction we're going with the project is headless and all about telemetry. If we could simply emit the transaction record as a Loki log, we would pretty much need nothing else in order to use it in HOMER 10 and Grafana already.

The change in heplify-server to send logs would be sufficient as there are no ui and api components.

lmangani avatar Aug 31 '23 11:08 lmangani

@lmangani hi, it means I need to migrate this feature to logs page?

silver0511 avatar Sep 05 '23 10:09 silver0511

@silver0511 that's one way to do it, but not necessarily the only one. Are you familiar with our latest homer 10 preview? https://github.com/sipcapture/homer-docker

Basically as long as a type emitted towards the Loki receiver is generated, this would work out of the box. You can use a log type if you want no restrictions, but make sure the body can be parsed with logfmt or json, etc

lmangani avatar Sep 05 '23 10:09 lmangani

@silver0511 that's one way to do it, but not necessarily the only one. Are you familiar with our latest homer 10 preview?

@lmangani not yet, we are currently focusing on the development of homer 7 :)

silver0511 avatar Sep 05 '23 10:09 silver0511

@silver0511 that's one way to do it, but not necessarily the only one. Are you familiar with our latest homer 10 preview? https://github.com/sipcapture/homer-docker

Basically as long as a type emitted towards the Loki receiver is generated, this would work out of the box. You can use a log type if you want no restrictions, but make sure the body can be parsed with logfmt or json, etc

i see, but now we are using this feature in the visual interface, which is very helpful for us,by the way, is there a big difference between homer 10 an 7

silver0511 avatar Sep 05 '23 10:09 silver0511

Less moving parts. There's no custom UI in HOMER 10 everything is done in grafana with a Flow plugin for HEP and using Loki, Prometheus and Tempo for logs, metrics and traces respectively. You can run our demo to check out the new experience we're building including the flow display.

lmangani avatar Sep 05 '23 10:09 lmangani

thanks, i will try it :)

silver0511 avatar Sep 06 '23 08:09 silver0511