发现网易公开课 Android 版不小心在 release apk 的时候把 .git 目录打包了进去

[pexcn]

几年前就发现他们的 apk 里面有个 .git 目录，今天闲着没事就拿出来翻一翻。 然后发现从不知道那个版本开始，一直到 3.2.9 版，apk 里面都有个 .git 目录。

然后我就把它的 source checkout 了出来，看了看，感觉写得不错。 只是 code style 有待改进，可以学习一个~

顺手放到了 GitHub: https://github.com/pexcn/com.netease.vopen

技术部的人联系我了，他们说是因为牵扯到一些内部信息，不方便公开。所以我将把代码撤掉~

[lusyoe]

楼主这样不道德吧，直接就把人家公司的内部产品代码公开出来了~

[techmoe]

估计再过一会网易的邮件就要到 Livid 或者 lz 的手里了

[haozhang]

public static final String QQ_APP_KEY = "801125947";
public static final String QQ_APP_SECRET = "0cf77f62b7b2288ac72bb38045d3a6b9";
我觉得网易的人要重新申请 appkey 了。→_→

[mokeyjay]

确实不太好

[metorm]

我很好奇，这个 git 仓库体积是多少呢？

[kindjeff]

逆向软件是违法的吧～～楼主赶紧细软跑

[josephchiang]

gh 上 33c5bcd on 12 May 2015 什么鬼我穿越了? ? ? 标题党还是旧闻放送？

[RyougiShiki]

习惯用 ide 自带的图形界面，比如 pycharm，提交前的清单一目了然 ，把不需要提交的文件勾去掉。而用命令行的小伙伴偶尔把修改的 settings.py .env 等文件提交上去。

[RyougiShiki]

为什么外人可以 checkout 下来，他们没设置账号权限吗

[pexcn]

@kindjeff @RyougiShiki 这不是逆向，他们把 .git 目录打包进 apk 了…

[pexcn]

@josephchiang 这是 2015 年的版本了，所以 commit 是 2015 年的

[pexcn]

@metorm 这只是客户端的代码，大概三万多行，而且不包含 resource 的

[pexcn]

@haozhang 他们在 3.3.0 版本就已经修复了这个问题，估计 key 早就更新了吧～

[pexcn]

@lusyoe 没有…是他们自己公开的，我只是感觉代码不错，发出来，感觉可以学习一个而已…

[zhidian]

可耻地 fork 了。==

[stanjia]

网易"公开"课

[pexcn]

@stanjia 😂😂😂

[kx5d62Jn1J9MjoXP]

真棒，看看大厂到底是个什么水平

[odirus]

我对 QQ 写成 Qq 这种情况一直比较反感，希望不是我一个人，这样写总感觉怪怪的

[Khlieb]

真·网易公开课

[Technetiumer]

真 · 网易「公开」课 😂😂😂

[lsmgeb89]

这违法的吧

[maplerecall]

emmm，这版本好早了，不过明天还是给安卓开发那边说下好惹_(:з」∠)_

[yyfearth]

@odirus 很合理的编码规范啊 严格遵守驼峰 就算是专有名词 我记得貌似 Google 的规范就是这样
你想想那么多专有名词 如果保持原样写在一起根本分不出来
QQ IP 之类的还好 iOS iPhone macOS 就更糟糕了

以前在 Apple 的时候 我们小组没有统一编码规范 有人坚持要保证这些专有名词的大小写正确
而我和几个新来的又坚持严格遵守驼峰 于是 iOS IOS Ios 混杂 代码没法看了

看看下面的这些方法 你觉得是那个好呢？

is iOS IPSec Tunnel Ready for SaaS API:
isiOSIPSecTunnelReadyforSaaSAPI
isIosIpsecTunnelReadyForSaasApi

get QQ FQDN IP in JSON:
getQQFQDNIPinJSON
getQqFqdnIpInJson

[binux]

@yyfearth 第一种好，因为有的专有名词小写之后我就认不得了，眼生

[beginor]

Java 还是太罗嗦， 哪位有兴趣翻译成 kotlin ？

[tamlok]

@yyfearth 三个字以内的缩写保持大写，以上的按照驼峰

[googlebot]

给 lz 点赞，赶紧 fork

[thomaswang]

这个包该有多大啊

[redtea]

JavaBean 如果不注意大小写问题的话，会给自己挖坑。

[dong3580]

好奇，只靠.git 文件怎么 checkout 源码呢？

[hjc4869]

@dong3580 直接把 .git 放到一个目录里执行 git reset --hard 就拿到当前 branch 最新的了

[BOYPT]

(这就是你们 apk 那么大的原因？？

[odirus]

@yyfearth 嗯，解释得有理有据，已点感谢

BTW，你在 apple 的时候，iOS 都能乱写？要是我的话，我受不鸟

[toono]

重磅😂

[rswl]

fork 一下

[bkmi]

@pexcn 这怎么不是逆向了，你从安装包里弄出来的，就是逆向

[pexcn]

@BOYPT 3.2.9 版本的 apk 并不大，5MB 左右而已

[21grams]

先 fork 了再说

[yyfearth]

@odirus 显示出来的字符串 iOS 当然必须是对的 会有专人翻译和校对
但是代码里面的变量 常量 类和方法名 当然是按照代码规范来写
但是我们不是核心团队所以没有统一的代码规范
我写 JS PHP 的时候

常量全大写 IOS_VERSION 总不会写成 iOS_VERSON 吧

其它情况就很多了 有 IOSClass IosClass IPhoneClass IphoneClass 甚至 iOSClass iPhoneClass 的
getIOSVersion getIosVersion 甚至 getiOSVersion 的

这些用户是看不见的 而且我也不写开放的 API 所以产品和管理不会管的

[laoyur]

说先 fork 为敬的人，你们也太...















naive 了，一旦被 DMCA，你们的 fork 还能留？又不是没有先例！
所以，我直接 download zip😝

[bk201]

这实习生写的吧

[odirus]

@yyfearth 好的，感谢解惑

[pexcn]

@laoyur 其实就算被 DMCA, 也可以从它的 3.2.9 版的 .git 目录里再 checkout 出来的...

[corvofeng]

真的是可耻的 fork 了， 也是学习一下吧

[zzczzc]

看一下大产的水平

[VYSE]

说不定故意开源呢,赞一个支持开源

[FanWall]

网易大概会怎么应对？

[pexcn]

@FanWall 应该无所谓了，毕竟是两年前的代码了

[yefuchao]

可耻的 Git clone 到本地了= P

[zhuangtongfa]

几年前的代码了

[devzero]

从 3.0.1 到 3.2.9，中间 14 个版本都有.git 目录

[voocel]

网易这开源精神值得赞扬

[Chyroc]

把 git 目录放到 git，那么不是永远有更改的代码？

刚 commit，git 目录又改了

[maplerecall]

嗯……目前评估影响不大的样子，里边东西都挺旧了，你们就把这当做一节公开课好咯 XD

[Kalv8n]

强行开源

[acoder2013]

在课程详情中，增加北京广告信息 。。。

[Hzzone]

宝宝赶紧 fork 了一份...替楼主烧香。。。

[lilydjwg]

@yyfearth #24

is_iOS_IPSec_tunnel_ready_for_SaaS_API
get_QQ_FQDN_IP_in_JSON

同专有名词写小写不太认得出来。

[yukiww233]

opencourse 和 opensource 很接近啊
没准人家是故意的呢 233333

[Gothack]

刚看到了，然后刷新了一下，没了。。

[pexcn]

@Gothack 然而有很多 fork 的。。。（ Run...

[Trumeet]

來晚了，43 分鐘前 Removed （

[pexcn]

@Trumeet 😂不是有很多 fork 嘛。。

[llxx510200]

@laoyur 还能发我一份吗，，大佬