Skip to content
This repository was archived by the owner on Nov 27, 2023. It is now read-only.

Add support to sas authorization #958

Merged

Conversation

polkfarody
Copy link
Contributor

@polkfarody polkfarody commented Nov 20, 2017

Piggy backing off @Shaked original Pull request. Made the required changes to get it running on our specific environment.

Main change was revolved around running composer update to revert the doctrine packages that require php >=7.

Also changed createWrapService to a static method to avoid a fatal "$this in non object context" error.

@msftclas
Copy link

msftclas commented Nov 21, 2017

CLA assistant check
All CLA requirements met.

Copy link

@Shaked Shaked left a comment

Choose a reason for hiding this comment

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

IMO it looks good, thank you for your help guys!

@sergey-shandar sergey-shandar merged commit 0d3ce5b into Azure:master Nov 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants