經過一個多月的等待我有幸成為Spring相關專案的Contributor

碼農小胖哥發表於2022-05-07
Spring

給開源專案尤其是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 獲取更多資訊

個人部落格：https://felord.cn

相關文章