[Nextcloud/android] 첫 컨트리뷰트

팀원들과 구글 공유 문서를 통해 Nextcloud/android 프로젝트의 패키지 분석을 진행했다.

이 과정 중에 발견한 오타 2개를 수정하여 Pull Request를 날렸다.

코드 주석에서 발견한 첫 번째 오타

기여 가이드에서 발견한 두 번째 오타

오픈소스에 처음 기여하는 것이라 그런지, 모든게 무서웠다..

특히 PR을 날린 후에 자동으로 품질검사가 진행되며, Fail 되었다고 알림이 왔는데 

10개의 테스트 중 3개가 fail 되었다.

첫 번째 QA 테스트는 비밀번호가 틀려서(?) 오류가 난 것 같다.

JavaKeyStore는 SSL 암호화를 위해 사용하는 키 저장 파일인데,

이 파일이 손상되었거나 암호가 틀려서 FAIL이 뜬 듯 싶다.

BUILD SUCCESSFUL in 4m 32s

35034 actionable tasks: 34 executed

351Failed to load signer "signer #1"

352java.io.IOException: Keystore was tampered with, or password was incorrect

353 at sun.security.provider.JavaKeyStore.engineLoad(JavaKeyStore.java:792)

354 at sun.security.provider.JavaKeyStore$JKS.engineLoad(JavaKeyStore.java:57)

355 at sun.security.provider.KeyStoreDelegator.engineLoad(KeyStoreDelegator.java:224)

356 at sun.security.provider.JavaKeyStore$DualFormatJKS.engineLoad(JavaKeyStore.java:71)

357 at java.security.KeyStore.load(KeyStore.java:1445)

358 at com.android.apksigner.SignerParams.loadKeyStoreFromFile(SignerParams.java:353)

359 at com.android.apksigner.SignerParams.loadPrivateKeyAndCertsFromKeyStore(SignerParams.java:239)

360 at com.android.apksigner.SignerParams.loadPrivateKeyAndCerts(SignerParams.java:181)

361 at com.android.apksigner.ApkSignerTool.getSignerConfig(ApkSignerTool.java:368)

362 at com.android.apksigner.ApkSignerTool.sign(ApkSignerTool.java:291)

363 at com.android.apksigner.ApkSignerTool.main(ApkSignerTool.java:84)

364Caused by: java.security.UnrecoverableKeyException: Password verification failed

365 at sun.security.provider.JavaKeyStore.engineLoad(JavaKeyStore.java:790)

366 ... 10 more

367Error: Process completed with exit code 2.

두 번째 오류는 DCO  테스트에서 오류가 발생했는데, 이는 Commit 할 때 서명을 안해서

발생하는 오류였다. 바로 수정하여 테스트를 통과할 수 있었다.

아래 명령어를 통해 서명할 수 있다.

 git commit --amend --signoff                                          

마지막으로 발생한 fail은 continuous-integration에서 발생했는데,

 찾아보니, 지속적으로 품질관리를 적용하는 프로세스를 실행하는 것으로, 모든 개발이 완료된 뒤에 품질관리를 하면 오류를 발견했을 때 너무 많은 비용이 드니까 통합할 때마다 CI 테스트를 진행하는 것이다.

나는 주석 속 오타와 md 파일의 오타를 수정한 것이므로 테스트 실패 원인을 해결하려고 하진 않았다

컨트리뷰터가 되었다 😍