※ 本文轉寄自 ptt.cc, 文章原始頁面
看板Soft_Job
標題

新手剛入職就發現有SQL injection漏洞 該回報嗎

時間
最新2023-05-06 12:47:00
留言227則留言,68人參與討論
推噓40 ( 488171 )
先簡單自我介紹一下 小弟自學+上課 學了一段時間的Java(大約三年) 偶爾也會上github對 開源專案發一些PR 也會看一些資安 資訊方面的相關議題 因為住在蠻偏遠的地區 所以直到 最近才有找軟體方面的工作 目前剛錄取一份接案公司的工作 主要語言是php 今天整天幾乎都在看code 但是跟github上開源的code比起來真的是又髒又亂 變數命名沒照規則 而且還用老舊的php5 例如2023年的案子裡面還有deprecated的meth od 更別說程式碼感覺都是貼來貼去的 拿以前的來貼改 檔案名稱 資料夾名稱也是都奇奇怪怪 整個專案裡面一大堆無關的code 看code看到後面越來越不對勁 一個簡單的or 1=1就看出來根本沒防sql注入 下班後用某工具對公司的一個案子試了一下 直接把所有db跟table還有資料直接撈出來 重點是全部案子 包括公司的一些資訊都在同一個server上可以直接撈 請問我該回報嗎 回報了不知道會不會被黑…(雖然我手上還有另一份offer 第一次在本版po文請見諒 -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 114.27.18.174 (臺灣) ※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1683045945.A.DF8.html

Re: 回文串

40227
> 新手剛入職就發現有SQL injection漏洞 該回報嗎
Soft_Job05/03 00:45

227 則留言

※ 編輯: jeff87218 (114.27.18.174 臺灣), 05/03/2023 00:48:30

t64141, 1F
如果另一個 offer 能接受不如就直接閃人吧

lee457088, 2F
回報為啥會被黑,語氣正常的話應該大家都很願意學習吧

hippo130, 3F
這些改善的工作就交給你了

s06yji3, 4F
趕快跑啊,還回報?

Mutibil, 5F
好典型新手的想法 開源 追求code完美簡潔blablabla

neo5277, 6F
快逃

Mutibil, 7F
都是接案公司了 當deadline逼近 誰還管樣樣都完美啊

chchang0820, 8F
老闆請你來幹嘛的?有問題不回報,上PTT發問,笑死

Mutibil, 9F
能趕快交付 拿到錢比較重要 你該了解當初這樣做的事由

chchang0820, 10F
先跟你上頭討論吧!怎麼會是先發文跟鄉民討論 笑死

brucetu, 11F
趕快換一家有點水準的公司

superpandal, 12F
最好用pdo傳參 php官方文檔很好 寫php最愜意的就是

superpandal, 13F
邊看文檔寫一寫發現這樣就寫好了

superpandal, 14F
文檔也有包 當然亂寫的肯定不會這樣做

superpandal, 15F
簡潔與安全和擴充性不是衝突的 我寫的是這樣 XD

superpandal, 16F
所以都不想用框架 什麼高並發原生其實也都可以

superpandal, 17F
解決 php本身就是對c語言的封裝 會寫了對了解c有幫助

superpandal, 18F
其它的語言都是研究個老半天 然後例如在jdk裡有些細

superpandal, 19F
節 而且更新php版本你應該考慮 如果你是oop狂人

Nonsense8, 20F
這種等級的技術債還是快逃吧,有礙成長,除非錢多真香

Nonsense8, 21F

Nonsense8, 22F
追求乾淨的程式跟新手老手沒關係,支持原po保持初衷,

Nonsense8, 23F
但也要考量各種外力影響,避免成為武痴

Nonsense8, 24F
至於要不要回報…很大程度取決於你們公司風氣,建議你

Nonsense8, 25F
多觀察和試探前輩跟同事的反應

superpandal, 26F
言歸正傳 回報要重構

Nonsense8, 27F
有可能大家都心裡有數但就是嫌麻煩而已。嗯不是每個人

Nonsense8, 28F
都覺得進步是好事…

superpandal, 29F
用vi/vim重構 不行就閃

superpandal, 30F
現在php xdebug斷點追縱也很方便

pirate12, 31F
快跑 這問題根本不該發生

CoNsTaR, 32F
某 Mutibil 這完全不是追求完美的問題吧

CoNsTaR, 33F
事情亂做也可以被你美化成"當初這樣做的事由"到底有沒有

CoNsTaR, 34F
底線啊

CoNsTaR, 35F
你這樣亂搞用的理由是"能趕快交付",我就問到底是亂搞還

CoNsTaR, 36F
是把事情做對能比較快交出能用的東西?

CoNsTaR, 37F
當然如果你的程度就是只能亂搞,沒日沒夜的亂搞花一堆不

CoNsTaR, 38F
必要的時間繞不必要的圈圈解不必要的 bug 最後總算生出一

CoNsTaR, 39F
個勉強能用的東西你大概會很自豪自己在 deadline 前生出

twgamerave, 215F
一改再改

dream1124, 216F
可以回報啊 剛好試試他們是怎麼對待新人和這類問題

dream1124, 217F
如果你沒講,那事情也許永遠沒機會改善

dream1124, 218F
另外,其他亂七八糟、髒亂等問題是真實世界常有的

dream1124, 219F
看亂的程度和能處理的資源多寡,有時只好先放著不管

viper9709, 220F
推原po的回應

akito117, 221F
正常也不會要你改,沒收入又要大改,又有可能有新的錯

akito117, 222F
誤,只能後來的案子多注意一點吧

new122851, 223F
寧願多加班不拿錢也要寫clean code. deadline可以延期

new122851, 224F
頂多賠錢

newnovice, 225F
惦惦or換公司

Darkword1987, 226F
我的話會先私下問前輩 不會直接往上捅 那當然如果

Darkword1987, 227F
得到的結論是擺爛的話我會考慮開始找下一份