Skip to content

Conversation

MEFSMART
Copy link

@MEFSMART MEFSMART commented Feb 10, 2021

Every branch from https://github.com/smartcoop/demo-nginx-auth-request - Connecter à l'aperçu repository for wich the name contains “authrequest/” will be automated deployed in 2 Docker containers.

These 2 Docker containers are:

  • 1 containing authentication API

  • 1 containing my VerySimpleApi project

The VerySimpleApi is available only if the user is connected.

To test, you can use the existing Docker containers.

They are available to:

Note: the URL contains “test” because my branch name is “authrequest/test”.

If you want to test an automated deployment, you can create another branch containing “authrequest/”. After pushing, the deployment is automatically launched.

To authenticate you, use only your trigram (nothing else) and your password. It’s working using ldap.

It isn't important to read everything about the VerySimpleAPI in this PR.

Copy link
Contributor

@AnthonyLammens-smart AnthonyLammens-smart left a comment

Choose a reason for hiding this comment

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

You forget to remove file From Api Folder (old api project)

Comment on lines +15 to +16
//_inMemoryUsers = new Dictionary<string, string>();
//_inMemoryUsers.Add("bob", "bob");
Copy link
Contributor

Choose a reason for hiding this comment

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

my bob :'(

Copy link
Author

Choose a reason for hiding this comment

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

RIP ;)

Comment on lines +10 to +13
# What was the purpose of this "exercice" ?

The purpose of this work is to show how to automatically deploy a .NET Core API project in a Docker container and how to make it accessible through a URL.

Copy link
Contributor

Choose a reason for hiding this comment

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

copy/paste from another project?

Copy link
Author

Choose a reason for hiding this comment

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

It comes from my VerySimpleAPI project ;)

@MEFSMART MEFSMART requested a review from thusc March 18, 2021 16:21
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