onvif-rs icon indicating copy to clipboard operation
onvif-rs copied to clipboard

added necessary fields for event pulling and an event example

Open ozdoganoguzhan opened this issue 1 year ago • 4 comments

Type Changes: Added some fields that are required to create a subscription for pulling messages with a filter, and also for pulling messages from that subscription.

New Example: Added an example on how to pull messages periodically from a specific camera. Tested on Dahua.

Screenshot_10

Solves #107

ozdoganoguzhan avatar Sep 26 '23 09:09 ozdoganoguzhan

Can you please rebase on main to see if CI is green?

DmitrySamoylov avatar Sep 28 '23 09:09 DmitrySamoylov

hi, how is it going?

asuper0 avatar Jan 15 '24 02:01 asuper0

The problem was in the test XML. I removed the envelope declarations and just left the responses and put the namespaces in them. Also moved the related tests to the end of tests file, to prevent conflict with the more recent commits to test file.

ozdoganoguzhan avatar Apr 03 '24 13:04 ozdoganoguzhan

error: useless use of vec!

This was fixed in main. Please, rebase again.

DmitrySamoylov avatar Apr 09 '24 12:04 DmitrySamoylov

Sorry for chipping in, also very interested in subscriptions & pull_messages. Really appreciate your work folks!

sqrtsanta avatar Apr 15 '24 14:04 sqrtsanta

Not sure if you receive notifications on failing CI jobs, but clippy is not happy now with some of the changes.

DmitrySamoylov avatar Apr 25 '24 10:04 DmitrySamoylov

everything should be ok now, thanks for your patience 😄

ozdoganoguzhan avatar Apr 25 '24 12:04 ozdoganoguzhan