Closed sswargam closed 4 years ago
Thanks for the PR, however, these changes break the existing API and will prevent applications that use this library from upgrading to a newer version without code changes. The library is currently being used for at least one commercial application that I'm aware of and many private implementations specific to various orgs. Requiring code changes on their part is a non-starter.
The applications that use the API would only invoke the ThreatModelFactory isn't it? We could leave the iParser method and add a new method for the InputStream. Let me know if you would like that.
Added support for Medical Device template
The iParser interface has changed to support InputStream. However, the ThreatModelFactory remains intact with added support for InputStream.