With the new rate limits implementation, we no longer need the initialIP column, and would like to drop it.
There's a default (zero) value for the column, so Boulder can safely stop writing it.
|
`initialIP` binary(16) NOT NULL DEFAULT '\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0', |
|
// Although deprecated, this column remains NOT NULL in the database, so |
|
// a value must still be provided. |
|
InitialIP: net.ParseIP("0.0.0.0").To16(), |
Boulder's code says a value must be provided because it's NOT NULL, but since there's a default, it should be OK I think.
Once there's no more code paths touching it, we will drop the column.
With the new rate limits implementation, we no longer need the initialIP column, and would like to drop it.
There's a default (zero) value for the column, so Boulder can safely stop writing it.
boulder/sa/db/boulder_sa/20230419000000_CombinedSchema.sql
Line 203 in 9b3c882
boulder/sa/model.go
Lines 331 to 333 in 9b3c882
Boulder's code says a value must be provided because it's NOT NULL, but since there's a default, it should be OK I think.
Once there's no more code paths touching it, we will drop the column.