search

zeroc-ice / ice

All-in-one solution for creating networked applications with RPC, pub/sub, server deployment, and more.
https://zeroc.com
GNU General Public License v2.0
2k stars 592 forks source link

Ignore Ice.Plugin.IceSSL for compatibility with 3.7 #2397

Open pepone opened 2 days ago

pepone commented 2 days ago

What will happen if the user keep this property in his config file? Do we have any validation of plug-in names with an Ice prefix?

Originally posted by @bernardnormier in https://github.com/zeroc-ice/ice/pull/2388#pullrequestreview-2154678837

We should review, but I think we should probably ignore the property.

externl commented 2 days ago

There are already lots of property changes: we're removing deprecated properties, old irrelevant properties, etc. I don't think we should specially ignore a single property just for version sharing.

IMO if you use Ice.Plugin.IceSSL with 3.8 you should get a warning, assuming we have check for Ice.Plugin.XXX