データ移行をしただけなのに…（起こってしまったメール誤配信） - Qiita

この記事は本番環境でやらかしちゃった人のアドベントカレンダー9日目の記事です。 https://qiita.com/advent-calendar/2019/yarakashi-production もう15年以上前の事なので記憶も定かではないところがありますが、ご容赦下さい。 当時の状況 当時自分は30人くらいの、孫請・曾孫請を中心に受託開発を行う小さなSIerに居ました。 この会社、自社製品も一応あるのですが売上のメインは圧倒的に受託開発で、 PHPやPerlでのガラケーサイトの開発やら、Javaや.NETを使った業務アプリケーションの開発、大手プロバイダシステムの開発保守など、わずか3年ほどの在籍期間でしたが、実に多彩な開発案件があったように思います。 プロジェクト内容 あるWebサイトのシステム移行でした。 Windows Server上に構築されたIIS+ASP+SQL Serv

[z1h4784]

これは気づかない。爆弾をセットした上司が全面的に悪い

[perl-o-pal]

「報連相がだいじ」って気づきを得たのに自分はその上司と口をきかないという趣。

[everybodyelse]

データベースにビジネスロジックを仕込む者には死を

[minamimonji]

「（客の担当名）抹殺」と書いたメールテスト文を見事に顧客に誤送信した元同僚は元気かなぁ〜（遠い目

[risouf]

この上司、「センス」ありすぎじゃない……？

[hasegawatomoki]

メール誤送信は本当に怖いので、最近は開発者のローカルや社内の環境は全部 mailcatcher でメール捕まえてる。

[cl-gaku]

絶対この上司他にも山ほどネタあるわ

[prograti]

検証環境にマスキングなしで会員情報の実データを入れようとしたという事かな？それはそれで問題があるような。

[aikawame]

メールの誤送信はダメージがでかいんよな…。開発環境でメール送っちゃったとかはあるあるなやらかしだけれども、これはかなりお気の毒度が高いと思う。

[yhachisu]

ヤバい人とは一緒に仕事をしない、という対策がほのめかされている気がする…

[northlight]

これは上司切腹モノ・・・

[shoh8]

“3. 目的の処理だけが実行されることを保証した環境をつくる”隠れコードを書くレコード /mailcatcherお世話になってます。mailtrap.ioも

[digo]

データ更新トリガーでサーバレス起動みたいなのちょっと流行ってるっぽいのでよい知見。

[knjname]

身内に敵がいるのは辛いな

[new3]

自分のやらかしというか人のやらかし…これで実はI田さんが後半に登録とかしていて出てきたりしたら胸熱。

[sds-page]

池田ァ！

[zuzu_sion]

PL/SQLあるあるですね。。可能ならばDDLを毎日差分管理すると良いかなと（DDLをSQLとして出力してGitにコミット。変更があれば通知するようにするとか）

[rryu]

実装担当でもなさそうな上司が勝手に実際にメールを送信するトリガーを仕掛けているとか罠以外のなにものでもない。

[Error401]

FOR EACH ROWなtriggerにビジネスロジック埋め込むの大好き

[ytRino]

いい怪談

[x100jp]

ブコメに良い知見が集まっているけど、この辺りは体系だってないから、やらかす"前"に学べるかは自主勉量と運次第なんだよなあ。自分の場合、運とはてブが良かった。大手だと、計画レビューで防げる可能性もあるけど

[shozzy]

うわー、これも怖い。そして、うちの委託先でもそんな感じの上司と部下の関係性を見ることがあって何度も指導しているが（以下略

[Lumin]

こんな一件があったら確かにまともに口も聞けなくなる。上司が人間に見えなくなると思う

[lorenz_sys]

"ビジネスロジックを分散させない" そう。それがすごく大事。いろんな責務をアプリやミドルウェアに分散させてるチーム(技術者)はまだまだ経験不足。開発時はいいが運用フェーズ(数年後)でひどい目に会うことになる。

[takhasegawa]

まずメールアドレスのようなセンシティブなデータは開発環境ではマスキングする。あと、開発環境のネットワークは外部から分離するか、できなければせめてFWでsmtpのポートを塞ぐ。報連相頑張るみたいな根性論はダメ

[stealthinu]

insertのtriggerでメール送る設定仕込んで誰にも言わなかったとか…　絶対事故るのわかるだろうに、なぜ？？

[likk]

今はどうなのか知らないけど、15年ぐらい前で Oracle 案件だと確かに trigger にビジネスロジック含まれる案件ごろごろしてたなーと。今は mysql しか触れてないので trigger 書こうなんて思わないけど。 / 開発環境はmaskするよね

[sudo_vi]

いい話

[jumpeim37]

これは気づかないなぁ

[deep_one]

そういうところでメール出していいのはログ代わりのシステムメッセージだけだろ…／コメントを見て。最終テストは実データでやるだろう。実データには魔物が住んでいるからな…。

[swdrsker]

移行テスト前の改変なんて1行の改変でも横展開なり影響確認なり必要なのに

[suika3417]

DBに余計な仕事はさせてはいけない…

[oktnzm]

そんな送信量制御もできないとこに書くもんじゃないでしょ。上司が100%悪いでいい。

[tohima]

何もしないでいてくれた方がマシという好例

[tyoro1210]

『この日不在の）上司がOracle上にPL/SQLでメール配信の処理を書』『誰にも言わずに…』『この日以降、転職するその日まで上司とはまともに口を利かなくなった』

[hoppie]

大物だ

[Kil]

「PL/SQLでメール配信の処理を書き」その発想はなかった。PL/SQLからメールって飛ばせるんだ……。

[yarumato]

“おれはSQL ServerからデータをOracleに移行していると 思ったらいつのまにか4,000人のユーザーにメールを送信していた。この日不在の上司が前日Oracle上にPL/SQLでメール配信の処理を書きTrigger設定していた。誰にも言わずに”

[rgfx]

oh...

[zarathustra9696]

ほうれんそう