Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Easy Auth local settings file to git ignore, project template #770

Closed
wants to merge 1 commit into from

Conversation

glennamanns
Copy link
Contributor

This is part of a larger goal to enable Easy Auth to run locally. The relevant Azure Functions CLI PR is here: Azure/azure-functions-core-tools#789

The CLI PR is currently blocked/waiting on the Easy Auth Linux+Windows combine, and this templates PR should be merged about the same time as that one.

@soninaren
Copy link
Member

soninaren commented Oct 10, 2018

@glennamanns , I don't think this is the right change. We shouldn't be adding any additional files to the default template which is an opt in feature. Instead this should be a CLI command like proxies. Function proxy create --name <yourproxyname>.

@glennamanns
Copy link
Contributor Author

Closing PR based on @soninaren's comment. I will instead programmatically add local.middleware.json to the Function's .csproj during the create-aad command.

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.

2 participants