給開源項目尤其是Spring這種知名度高的項目貢獻代碼是比較難的，起碼胖哥是這麼認爲的。有些時候我們的靈感未必契合作者的設計意圖，即使你的代碼十分優雅。
我曾經給Spring Security提交了一個我認爲非常重要的一項優化，和作者溝通了幾十個來回無法說服他。人家說留了抽象接口，你覺得不對自己實現一個就是了，默認的就是能用而已，無奈我就放棄了這個PR。
想不到在Spring Authorization Server中我抓住了一個能夠絕對被接受的機會，經過一個多月的等待，最終我的PR被接受了。
我在編寫Spring Authorization Server相關教程中，調用授權服務器元信息端點時不小心後面帶了個錨點符號#，類似http://issuer.felord.cn/#，報了一個請求錯誤，在追蹤源碼的時候我發現對這個url是進行過校驗的，出現了這種非預期的結果說明規則一定有漏洞。於是我查閱了相關RFC的定義，找到了依據RFC8414。
這個規範的實現沒有什麼技術含量，我如果告訴你，你也能做得到，甚至做得比我好，所以只要實現了，PR被接受應該是跑不了的事情，甚至可以半場開香檳。在0.3.0版本中這個優化將會實裝。對這個過程，我簡單總結了一下分享出來 ，或許對希望參與開源貢獻的你有所幫助。
代碼風格
首先是代碼風格，開源項目都有一些代碼風格要求，我之前有很多東西模仿(抄)了Spring的設計，所以我在實現規範的時候特別注意了這一點，命名風格、縮進等等都反覆推敲。
理解項目的架構思想
在整個PR過程中，我出現的問題主要在這裏。雖然我最初的實現執行起來沒有問題，完全在生命週期之內，但是顯然它的位置不是最佳。比如參數的校驗你可以寫在控制層，也可以寫在服務層，結果一定是沒問題的，但是過程可能有問題。作者和我溝通了一番，表明了他的設計思路，很快我就糾正了這個問題。
單元測試非常重要
實現代碼確實不多，單元測試的代碼幾乎超過了其20倍。我們平時寫個main方法簡單sout打印一下就完事了，對於開源項目可不一樣。尤其是這種知名開源框架，需要你測試實現的方方面面。如果你想爲開源項目貢獻代碼，常用的斷言、Mock插樁都要會，這個胖哥的博客felord.cn都有相關的介紹。
Git規範
Git 提交規範胖哥也多次強調了它的重要性，在大型開源項目中遵循一些提交規範也是一個基本要求。一些Pull Request的規則、代碼Review流程也要熟悉，這些功夫都在於平時。Git你得熟練。Spring要求最小功能提交，怎麼說呢？你的實現必須是一個最小粒度的完整單元，你不能在一個提交中既有增強優化（enhance）還有新特性（feature）。而且你的提交必須壓縮到一個git commit內，即使被打回去修改。要多用
git commit --amend -m '你的提交信息'
在一些不得已的時候，git rebase和squash commits或許能幫到你。
但是千萬不要去搞什麼Git原理底層，把工具用熟練就夠了。
要善於發現，敢做敢爲
最重要的一點，你要善於發現，比如下面這個bug（現在還沒有徹底修復），如果我說它是個bug，你一定會大喫一驚，我當時也是一樣的。
public DefaultSecurityFilterChain(RequestMatcher requestMatcher, List<Filter> filters) {
if (!filters.isEmpty()) {
logger.info(LogMessage.format("Will not secure %s", requestMatcher));
}
else {
logger.info(LogMessage.format("Will secure %s with %s", requestMatcher, filters));
}
this.requestMatcher = requestMatcher;
this.filters = new ArrayList<>(filters);
}
3.1版本的log邏輯居然錯到5.6版本！而修復它只需要改下位置，這個bug我媽應該也會改。但是沒有人發現和質疑它，直到最近有人提交了修復（5.7版本會修復這個bug）。
所以你要善於發現一些問題和細節，同時也要敢於質疑，即使質疑錯了又不會損失什麼。
要有專長
我和大家一樣，平常也是什麼都學，沒辦法！做原創技術博主太容易內容焦慮了。但是我刻意保證了一些東西去深入學習，有些東西必須成爲你的獨門絕技，要一精多專，這樣你在專長的領域可能有更多的機會。
TODO
對於成爲開源貢獻者只是一個開始，更多的挑戰還在後面。有人問我最近爲什麼更文少了！這裏也講一下我最近在做的事。
首先，因爲疫情我被隔離在家辦公了。
其次，還有幾個開源貢獻正在進行中。
還有，我正在做一個基於Spring Authorization Server的開源項目Id Server，你可以認爲是一個極簡版的keycloak，甚至最初的功能比較簡陋，layui都是先學的，界面我也做不好。放個截圖吧。
做這個東西很有挑戰，前端真是太難了！目前已經初具雛形，不日將放出預覽版，也歡迎各位同學參與貢獻。
關注公衆號：Felordcn 獲取更多資訊