Skip to content

Conversation

@QilongTang
Copy link
Contributor

Purpose

Enable NTLM authenticator for https://jira.autodesk.com/browse/DYN-2087.

Given we will need a branch build to pass to support, @mjkkirschner do you think we can publish Greg with this change on nuget and then branch Dynamo using that or there is a diff way

Declarations

Check these if you believe they are true

  • The codebase is in a better state after this PR
  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.

Reviewers

@mjkkirschner

FYIs

(FILL ME IN, Optional) Names of anyone else you wish to be notified of

@mjkkirschner
Copy link
Member

mjkkirschner commented Jan 6, 2021

yeah we can do that - or the simplest alternative may be to push your custom build of greg to a custom dynamo branch and reference it as a dll directly.

@QilongTang
Copy link
Contributor Author

@reddyashish @zeusongit @mjkkirschner The customer support suggests merging this and validate this ourselves. I suppose once I merge this, a newer version of Greg will be published and I will update my PR in Dynamo side to integrate that.

@QilongTang QilongTang merged commit c49b2fa into master Jun 22, 2021
@QilongTang QilongTang deleted the ntlm branch June 22, 2021 15:34
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