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.

Sign up for GitHub

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

Jump to bottom

API Key Feature #932

Merged
merged 7 commits into from
Jan 28, 2020
Merged

API Key Feature #932

merged 7 commits into from
Jan 28, 2020

Conversation

AmaliMatharaarachchi
Copy link
Contributor

@AmaliMatharaarachchi AmaliMatharaarachchi commented Jan 14, 2020 •
edited

Purpose

This will add the API Key support to Micro gateway. API Key authorization is introduced as well as self API Key Issuing capability

Issues

Fixes #897
Fixes #940
Fixes #1093

Automation tests

  • Unit tests added: Yes
  • Integration tests added: Yes

Tested environments

MacOS catalina
java version "1.8.0_201"

Maintainers: Check before merge

  • Assigned 'Type' label
  • Assigned the project
  • Validated respective github issues
  • Assigned milestone to the github issue(s)

@AmaliMatharaarachchi AmaliMatharaarachchi mentioned this pull request Jan 14, 2020
Closed
4 tasks
praminda
praminda reviewed Jan 21, 2020
View reviewed changes
components/micro-gateway-cli/src/main/java/org/wso2/apimgt/gateway/cli/model/config/APIKey.java Show resolved Hide resolved
...cli/src/main/java/org/wso2/apimgt/gateway/cli/model/template/service/BallerinaOperation.java Outdated Show resolved Hide resolved
...cli/src/main/java/org/wso2/apimgt/gateway/cli/model/template/service/BallerinaOperation.java Show resolved Hide resolved
...cli/src/main/java/org/wso2/apimgt/gateway/cli/model/template/service/BallerinaOperation.java Outdated Show resolved Hide resolved
...y-cli/src/main/java/org/wso2/apimgt/gateway/cli/model/template/service/BallerinaService.java Outdated Show resolved Hide resolved
...nts/micro-gateway-core/src/main/ballerina/src/gateway/handler_providers/api_key_provider.bal Outdated Show resolved Hide resolved
...nts/micro-gateway-core/src/main/ballerina/src/gateway/handler_providers/api_key_provider.bal Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/listeners/api_gateway_listener.bal Outdated Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/listeners/api_gateway_listener.bal Outdated Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/listeners/api_gateway_listener.bal Outdated Show resolved Hide resolved
@praminda praminda added this to In progress in MGW-3.1.0 via automation Jan 21, 2020
@AmaliMatharaarachchi
Copy link
Contributor Author

@praminda Fixed above review suggestions with 59daeab Thanks.

@AmaliMatharaarachchi
api key auth support and api key issue feature
4fb2461
@AmaliMatharaarachchi
Merge remote-tracking branch 'upstream/master' into my-jwt-r3
296bad4 
# Conflicts:
#	components/micro-gateway-cli/src/main/resources/templates/listeners.mustache
#	components/micro-gateway-core/src/main/ballerina/src/gateway/filters/subscription_filter.bal
#	components/micro-gateway-core/src/main/ballerina/src/gateway/listeners/api_gateway_listener.bal
#	components/micro-gateway-core/src/main/ballerina/src/gateway/listeners/api_gateway_secure_listener.bal
#	distribution/resources/conf/micro-gw.conf
praminda
praminda reviewed Jan 23, 2020
View reviewed changes
components/micro-gateway-cli/src/main/resources/templates/tokenServices.mustache Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/handlers/api_key_handler.bal Outdated Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/listeners/api_gateway_listener.bal Outdated Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/utils/api_key_utils.bal Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/utils/api_key_utils.bal Outdated Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/utils/api_key_utils.bal Outdated Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/utils/api_key_utils.bal Outdated Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/utils/api_key_utils.bal Outdated Show resolved Hide resolved
components/micro-gateway-core/src/main/ballerina/src/gateway/utils/api_key_utils.bal Outdated Show resolved Hide resolved
tests/src/test/java/org/wso2/micro/gateway/tests/apikey/APIKeyTestCase.java Show resolved Hide resolved
@AmaliMatharaarachchi
Copy link
Contributor Author

@praminda Fixed above review suggestions in 74736b7

@AmaliMatharaarachchi
Merge remote-tracking branch 'upstream/master' into my-jwt-r3
49a919a 
# Conflicts:
#	distribution/resources/conf/micro-gw-default.conf
#	distribution/resources/conf/micro-gw.conf
@Rajith90 Rajith90 added this to the 3.1.0-m1 milestone Jan 28, 2020
@Rajith90 Rajith90 merged commit d875a03 into wso2:master Jan 28, 2020
MGW-3.1.0 automation moved this from In progress to Done Jan 28, 2020
@AmaliMatharaarachchi AmaliMatharaarachchi mentioned this pull request Jan 29, 2020
Merged
4 tasks
@AmaliMatharaarachchi AmaliMatharaarachchi mentioned this pull request Feb 7, 2020
Merged
4 tasks
This was referenced Mar 2, 2020
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ruks ruks ruks left review comments

@menakaj menakaj menakaj left review comments

@praminda praminda praminda approved these changes

@Rajith90 Rajith90 Rajith90 approved these changes

Assignees
No one assigned
Labels
Docs/Has Impact Priority/Normal
Projects
No open projects
MGW-3.1.0
  
Done
Milestone
3.1.0-m1
5 participants
@AmaliMatharaarachchi @ruks @praminda @menakaj @Rajith90