復制粘貼出來的悲劇----spring實現文件下載和HttpStatus.CREATED

今天真是被自己的懶惰和復制粘貼給坑慘了...

網上有這么一個spring下載文件的最佳實踐:

@RequestMapping("download")  
public ResponseEntity<byte[]> download() throws IOException {  
    HttpHeaders headers = new HttpHeaders();  
    headers.setContentType(MediaType.APPLICATION_OCTET_STREAM);  
    headers.setContentDispositionFormData("attachment", "xxx.txt");  
    return new ResponseEntity<byte[]>(FileUtils.readFileToByteArray(getDictionaryFile()),  
                                      headers, HttpStatus.CREATED);  
}

可以看到返回狀態給的是HttpStatus.CREATED(201),這個當時想當然的認為既然是下載文件,肯定是新創建,所以create也沒什么,也就沒進一步仔細查詢一下201狀態碼的意義.然后呢這段代碼實際上在谷歌和火狐上執行是一點問題沒有的,所以一直以來就這么復制粘貼復制再粘貼...直到今天發現這段代碼實際上在ie上是有問題的.

然后折騰了一下午加上一晚上,一直以為是MediaType的問題,換了n種方式依然未果.因為我知道這里返回HttpStatus.OK也是可以,所以只是麻木的各種嘗試把它換成OK(200),結果在ie下真就好了.這才想起來仔細查詢一下HttpStatus.CREATED(201)的含義,大體含義如下:

服務器已經創建了文檔，Location頭給出了它的URL。

源碼中給的參考地址已經無法訪問,所以只能簡單的百度一下http狀態碼,上面只是其中的一種解釋,但是大體都是說服務器已經創建了文檔,這個實際上和下載文件沒半點關系..

所以正確的寫法應該是:

@RequestMapping("download")  
public ResponseEntity<byte[]> download() throws IOException {  
    HttpHeaders headers = new HttpHeaders();  
    headers.setContentType(MediaType.APPLICATION_OCTET_STREAM);  
    headers.setContentDispositionFormData("attachment", "xxx.txt");  
    return new ResponseEntity<byte[]>(FileUtils.readFileToByteArray(getDictionaryFile()),  
                                      headers, HttpStatus.OK);  
}