Skip to content

Feature/con 194 js - copyright updates and add logging section#53

Merged
rkorte merged 2 commits intodevelopfrom
feature/CON-194-js
Apr 7, 2021
Merged

Feature/con 194 js - copyright updates and add logging section#53
rkorte merged 2 commits intodevelopfrom
feature/CON-194-js

Conversation

@rkorte
Copy link
Copy Markdown
Contributor

@rkorte rkorte commented Apr 2, 2021

No description provided.

@rkorte rkorte changed the base branch from master to develop April 2, 2021 18:53
@rkorte
Copy link
Copy Markdown
Contributor Author

rkorte commented Apr 2, 2021

Do any of these issues need to be fixed

  • Some modules are not getting built (see here)
  • Does <participant_qos> tag update to domain_participant_qos (see CORE-11069) affect Connector?
  • The new feature about disposed instances says "Instance handles are not exposed" in python version, but not js version. Should they be the same?

All rights reserved. |br|
Printed in U.S.A. First printing. |br|
November 2019. |br|
April 2021. |br|
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought we only updated the (c) year when the product changed in a major fashion, but not for normal releases.

The security features of this product include software developed by the
OpenSSL Project for use in the OpenSSL Toolkit (http://www.openssl.org/).
OpenSSL Project for use in the OpenSSL Toolkit (http://www.openssl.org/).
his product includes cryptographic software written by Eric Young
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
his product includes cryptographic software written by Eric Young
This product includes cryptographic software written by Eric Young

However, Connector doesn't include any security features, Connext Pro does.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alexcamposruiz, what do you recommend here? Should I delete the whole "The security features of this product ..." paragraph (keep in mind it was there even in 1.0.0, although without my additions here)?

According to Legal, the sentences I added have to accompany that first sentence from now on, and I have made that change in all products. Per Legal, it's OK if the product doesn't even use security features, having it there "just in case" or even when not needed is OK, in order to product just one copyright statement that works for all books.

Given that, we could, for Connector:

  • delete the whole paragraph since Connector doesn't have security features
  • keep it since it interoperates with Pro, which can use security features
  • rewrite it to say something like, "The security features of RTI Connext include ..." if it just feels too strange to include this statement for Connector (I would want to run that by Legal, though)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It makes sense to keep it so we have a single copyright statement.

@alexcamposruiz
Copy link
Copy Markdown
Collaborator

Do any of these issues need to be fixed

  • Some modules are not getting built (see here)
  • Does <participant_qos> tag update to domain_participant_qos (see CORE-11069) affect Connector?
  • The new feature about disposed instances says "Instance handles are not exposed" in python version, but not js version. Should they be the same?

@samuelraeburn can you look into these issues?

@samuelraeburn
Copy link
Copy Markdown
Contributor

(Responses inline)

Do any of these issues need to be fixed

  • Some modules are not getting built (see here)

See attached screenshot, the links in that google doc are dead now, but for me locally all classes are built for both APIs. Can you check if it is still an issue with the latest doc builds?
Screenshot from 2021-04-06 08-40-54

  • Does <participant_qos> tag update to domain_participant_qos (see CORE-11069) affect Connector?

It does affect Connector, but I think I already fixed it. Have you found some instances of participant_qos that I have missed?

  • The new feature about disposed instances says "Instance handles are not exposed" in python version, but not js version. Should they be the same?

I have committed this change to this branch

@rkorte rkorte merged commit 0879842 into develop Apr 7, 2021
@rkorte rkorte deleted the feature/CON-194-js branch April 7, 2021 20:09
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.

3 participants