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.

Sign up for GitHub

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

Jump to bottom

Append TOI: Info exchanged during unsecured discovery #102

Closed
d-gregorczyk opened this issue Sep 19, 2022 · 6 comments · Fixed by #179
Closed

Append TOI: Info exchanged during unsecured discovery #102

d-gregorczyk opened this issue Sep 19, 2022 · 6 comments · Fixed by #179
Assignees
@PeterKranich @JavierEspina @d-gregorczyk
Labels
SDPi Specification Content to be added to the specification SDPi Topic of Interest Affects a Topic of Interest (ToI) SDPi Volume 2 Affects SDPi Volume 2

Comments

@d-gregorczyk
Copy link
Collaborator

See also: https://confluence.hl7.org/display/GP/Topic%3A+Info+exchanged+during+unsecured+discovery
@d-gregorczyk d-gregorczyk added SDPi Specification Content to be added to the specification SDPi Topic of Interest Affects a Topic of Interest (ToI) SDPi Volume 2 Affects SDPi Volume 2 labels Sep 19, 2022
@ToddCooper
Copy link
Collaborator

Update related sdpi-supplement-issues.adoc item when complete, indicating where the updates were made or the branche(s) that captured the updates, as well as moving from OPEN to CLOSED lists if appropriate.

@ToddCooper
Copy link
Collaborator

Note: @PeterKranich should do the PR review.

@ToddCooper
Copy link
Collaborator

From SDPi 1.0 Workshop: Info exchange during unsecured discovery: will be added to volume 3 general part (Todd) and volume 2 (David), Peter will review. David will ask Michael Faughn for missing participant / MDS type codes.

@ToddCooper
Copy link
Collaborator

@d-gregorczyk - NOTE: Todd will ask Michael Faughn for missing participant / MDS type codes (vs. the workshop notes).

@ToddCooper
Copy link
Collaborator

DECISION (2022.10.18): Move the "MDS" type codes of THIS topic to the existing topic:

 [Topic: System Types Nomenclature](https://confluence.hl7.org/display/GP/Topic%3A+System+Types+Nomenclature)

specifically move the section: Central SDC Service Participants to the System Types topic +
Mark the info exchanged topic as "resolved"

d-gregorczyk added a commit that referenced this issue Oct 20, 2022
@d-gregorczyk
#102 Intermediate result.
d712e8b
d-gregorczyk added a commit that referenced this issue Oct 27, 2022
@d-gregorczyk
#102 Intermediate result.
bdbd88b
d-gregorczyk added a commit that referenced this issue Oct 28, 2022
@d-gregorczyk
#102 Update cookbook to include identifier type "example".
dbcfc2d
d-gregorczyk added a commit that referenced this issue Oct 28, 2022
@d-gregorczyk
#102 Move contents to seperate files; add production specification an…
254d42c 
…d attribute usages.
d-gregorczyk added a commit that referenced this issue Oct 28, 2022
@d-gregorczyk
#102 Remove underlines from links to improve readability.
5cc3ee9
d-gregorczyk added a commit that referenced this issue Oct 28, 2022
@d-gregorczyk
#102 Rework directory structure, review of the contents, add glossary…
8a66e71 
… terms.
d-gregorczyk added a commit that referenced this issue Nov 1, 2022
@d-gregorczyk
#102 Change property prefix to term; add properties to glossary.
de29f61
d-gregorczyk added a commit that referenced this issue Nov 1, 2022
@d-gregorczyk
#102 Define term: Manufacturer.
4c812fa
d-gregorczyk added a commit that referenced this issue Nov 1, 2022
@d-gregorczyk
#102 Fix requirement levels; minor text changes.
7cfce5f
d-gregorczyk added a commit that referenced this issue Nov 1, 2022
@d-gregorczyk
#102 Add requirements level checker to Asciidoc tooling. Fail process…
84b920f 
… in case of an error.
d-gregorczyk added a commit that referenced this issue Nov 1, 2022
@d-gregorczyk
#102 Enforce requirement level error for CI run.
f60acb5
d-gregorczyk added a commit that referenced this issue Nov 1, 2022
@d-gregorczyk
#102 Fix requirement level issue.
264411f
@d-gregorczyk d-gregorczyk linked a pull request Nov 1, 2022 that will close this issue
102 append toi info exchanged during unsecured discovery #179
Merged
d-gregorczyk added a commit that referenced this issue Nov 1, 2022
@d-gregorczyk
#102 Solve merge conflicts.
f2732f9
d-gregorczyk added a commit that referenced this issue Nov 1, 2022
@d-gregorczyk
#102 Fix broken reference.
6a093d0
d-gregorczyk added a commit that referenced this issue Nov 3, 2022
@d-gregorczyk
#102 Solve merge conflicts.
97c4340
d-gregorczyk added a commit that referenced this issue Nov 9, 2022
@d-gregorczyk
102 append toi info exchanged during unsecured discovery (#179)
4b44103 
* #102 Intermediate result.

* #102 Intermediate result.

* #102 Update cookbook to include identifier type "example".

* #102 Move contents to seperate files; add production specification and attribute usages.

* #102 Remove underlines from links to improve readability.

* #102 Rework directory structure, review of the contents, add glossary terms.

* #102 Change property prefix to term; add properties to glossary.

* #102 Define term: Manufacturer.

* #102 Fix requirement levels; minor text changes.

* #102 Add requirements level checker to Asciidoc tooling. Fail process in case of an error.

* #102 Enforce requirement level error for CI run.

* #102 Fix requirement level issue.

* #102 Fix broken reference.
JavierEspina pushed a commit that referenced this issue Nov 10, 2022
@d-gregorczyk @JavierEspina
102 append toi info exchanged during unsecured discovery (#179)
19e600a 
* #102 Intermediate result.

* #102 Intermediate result.

* #102 Update cookbook to include identifier type "example".

* #102 Move contents to seperate files; add production specification and attribute usages.

* #102 Remove underlines from links to improve readability.

* #102 Rework directory structure, review of the contents, add glossary terms.

* #102 Change property prefix to term; add properties to glossary.

* #102 Define term: Manufacturer.

* #102 Fix requirement levels; minor text changes.

* #102 Add requirements level checker to Asciidoc tooling. Fail process in case of an error.

* #102 Enforce requirement level error for CI run.

* #102 Fix requirement level issue.

* #102 Fix broken reference.
JavierEspina added a commit that referenced this issue Nov 16, 2022
@JavierEspina @d-gregorczyk
@MartinKasp @ToddCooper @riechkathrin
96 append toi handling missed discovery messages (#186)
d821104 
* Issue #96. First draft proposal - further work and discussion needed

* Issue #96. Incorporated decisions of 10-Nov-2022 meeting - pending text re-shuffling and a reference

* #103 Add connect time delay section.

* #103 Add custom styles to default stylesheet; finish connect time algorithm description.

* #103 Add verification of requirement level attribute existence. Change requirement block name to sdpi_requirement.

* #103 Change title from "replacements & adjuncts" to "amendments & corrigenda"

* Add initial content for section "BICEPS Extension Model"

* #156 Apply suggestions from code review

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* #156 resolve comments from review

* #156 Apply suggestions from code review

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* #156 resolve comments from review

* Reorganized the Repository Use Case Folder + Added Latest SDPi 1.0 Use Case Documents

Added a folder for SDPi in the Use Cases folder
Moved existing "Use Case Inventory" files to the new SDPi folder
Added the latest use case files (SDPi 1.0 v1.5) to the directory

* #85 append dev-28

* #85 message semantics added

* #85 added suggestions from review

* #85 added suggestions from review

* #85 fixed dpws rendering

* Update vol2-clause-appendix-a-mdpws-dev-28-contextreport.xml

* #86 append dev-29

* #86 added suggestions from code review

* added suggestions from review

* #104 append toi mdib report retrofit

* Proposed changes in line with comments on issue & a few editorial tweaks

* Issue #104. Corrected type in R1007: expect->except

* #104 Tweak requirement texts. Transform replacements to regular SDPi requirements. Fix include flaw. Add missing reference to BPKP.

* #104 Add MDPWS specific part to the MDIB retrofit. Minor tweaks.

* Update tf2-ch-a-mdpws-mdib-report-retrofit.adoc

further tweaked R1005 as it was so efficiently expressed that it was hard for readers to comprehend

* Update tf3-ch-8.2.8-mdib-report-retrofit.adoc

Corrected Req nunbers from R1008 onwards (was repeated). Also made some editorial change.

* Update SDPi_Supplement/asciidoc/volume3/mdib-report-retrofit/tf3-ch-8.2.8-mdib-report-retrofit.adoc

* #89 append dev-34

* #89 added suggestions froms review

* Adding MDPWS Compression Option (issue#97)

* Adding MDPWS Compression Option (issue#97)

* little correction to correctly display R0006 in its box

* more formal re-phrasing of R0005 (citing actors)

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* replaced "client" for proper actor name

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* Once again, amended requirement text with proper actor names

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* Integrated 4 commits related to David's PR review

* #97 Minor fixes to the Asciidoc markup; change order of abbrev and bib ref.

* Support branch name in generated zip artifact

* 102 append toi info exchanged during unsecured discovery (#179)

* #102 Intermediate result.

* #102 Intermediate result.

* #102 Update cookbook to include identifier type "example".

* #102 Move contents to seperate files; add production specification and attribute usages.

* #102 Remove underlines from links to improve readability.

* #102 Rework directory structure, review of the contents, add glossary terms.

* #102 Change property prefix to term; add properties to glossary.

* #102 Define term: Manufacturer.

* #102 Fix requirement levels; minor text changes.

* #102 Add requirements level checker to Asciidoc tooling. Fail process in case of an error.

* #102 Enforce requirement level error for CI run.

* #102 Fix requirement level issue.

* #102 Fix broken reference.

* #86 append dev-29

* Issue #96. Re-shuffled text and added SOAP-over_UDP reference

* Update SDPi_Supplement/asciidoc/volume2/dev-23/tf2-ch-a-mdpws-dev-23.adoc

corrected "dot" before R2001

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* Update SDPi_Supplement/asciidoc/volume2/dev-23/tf2-ch-a-mdpws-dev-23.adoc

corrected "dot" before R2002

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* Update SDPi_Supplement/asciidoc/volume2/dev-23/tf2-ch-a-mdpws-dev-23.adoc

corrected "dot" before R2003

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* Corrected missing "dots" preceding requiremnts on mdpws-dev-24 file

* #96 Remove outdated file.

* fixed SOAP_over_UDP reference tag

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* editorial in mdpws-dev-24 (filter criterion example)

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* corrected SOAP-over-UDP reference tag in mdpws-dev-23

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* Tweaked recurring Hello justification paragraph

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* Improved paragraph on Hello message size (mostly style)

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* Style improvement in mdpws-dev-24.adoc

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>

* Replaced Hello message size req. W/ DPWS req. pointer

* note fix

* Added proper DPWS ref in mpdws-dev-23

* #96 Removed collapsible note between regular paragraphs; change vom plural to singular.

* Updated SDPi-supplement-issues for #96 and #97

Co-authored-by: David Gregorczyk <david.gregorczyk@draeger.com>
Co-authored-by: BBMAG\kaspmade <martin.kasparick@bbraun.com>
Co-authored-by: Martin Kasparick <112488805+MartinKasp@users.noreply.github.com>
Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>
Co-authored-by: Todd "AFC!" Cooper <ToddCooperAFC@gmail.com>
Co-authored-by: Riech <kathrinpia.riech@draeger.com>
Co-authored-by: riechkathrin <112853281+riechkathrin@users.noreply.github.com>
Co-authored-by: David Gregorczyk <david.gregorczyk@web.de>
@ToddCooper
Copy link
Collaborator

@JavierEspina -- In the master branch issues document, this item doesn't have any info as to what was updated in the supplement, though it is listed in the CLOSED section. Was there additional information that was going to be added to the closed issue list?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PeterKranich PeterKranich

@JavierEspina JavierEspina

@d-gregorczyk d-gregorczyk

Labels
SDPi Specification Content to be added to the specification SDPi Topic of Interest Affects a Topic of Interest (ToI) SDPi Volume 2 Affects SDPi Volume 2
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

102 append toi info exchanged during unsecured discovery IHE/sdpi-fhir
4 participants
@ToddCooper @PeterKranich @JavierEspina @d-gregorczyk