Skip to content

Commit

Permalink
refactor: cors 허용 url pattern 프로파일화 (#443)
Browse files Browse the repository at this point in the history
  • Loading branch information
tributetothemoon authored Aug 27, 2021
1 parent f41238e commit fa3b32e
Show file tree
Hide file tree
Showing 5 changed files with 25 additions and 3 deletions.
17 changes: 15 additions & 2 deletions backend/src/main/java/com/woowacourse/zzimkkong/WebConfig.java
Original file line number Diff line number Diff line change
@@ -1,17 +1,30 @@
package com.woowacourse.zzimkkong;

import org.apache.http.HttpHeaders;
import org.springframework.beans.factory.annotation.Value;
import org.springframework.context.annotation.Configuration;
import org.springframework.web.servlet.config.annotation.CorsRegistry;
import org.springframework.web.servlet.config.annotation.WebMvcConfigurer;

import java.util.List;
import java.util.stream.Collectors;
import java.util.stream.Stream;

@Configuration
public class WebConfig implements WebMvcConfigurer {
private final List<String> allowOriginUrlPatterns;

public WebConfig(@Value("${cors.allow-origin.urls}") String allowOriginUrlPatterns) {
this.allowOriginUrlPatterns = Stream.of(allowOriginUrlPatterns.split(","))
.map(String::strip)
.collect(Collectors.toList());
}

@Override
public void addCorsMappings(CorsRegistry registry) {
registry.addMapping("/**")
.allowedMethods("*")
.exposedHeaders(HttpHeaders.LOCATION);
// .allowedOriginPatterns("https://zzimkkong.com/");
.exposedHeaders(HttpHeaders.LOCATION)
.allowedOriginPatterns(allowOriginUrlPatterns.toArray(new String[0]));
}
}
3 changes: 3 additions & 0 deletions backend/src/main/resources/application-dev.properties
Original file line number Diff line number Diff line change
Expand Up @@ -24,3 +24,6 @@ cloud.aws.stack.auto=false

# svg converter
converter.temp.location=/home/ubuntu/zzimkkong/tmp/

# cors (delimiter == ',')
cors.allow-origin.urls=*
3 changes: 3 additions & 0 deletions backend/src/main/resources/application-local.properties
Original file line number Diff line number Diff line change
Expand Up @@ -31,3 +31,6 @@ cloud.aws.region.static=ap-northeast-2

# svg converter
converter.temp.location=src/main/resources/tmp/

# cors (delimiter == ',')
cors.allow-origin.urls=*
3 changes: 3 additions & 0 deletions backend/src/main/resources/application-test.properties
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,6 @@ cloud.aws.region.static=ap-northeast-2

# svg converter
converter.temp.location=src/main/resources/tmp/

# cors (delimiter == ',')
cors.allow-origin.urls=*
2 changes: 1 addition & 1 deletion backend/src/main/resources/config

0 comments on commit fa3b32e

Please sign in to comment.