I think that this should be constant. I also think that it should be defined near its usage as it is not public API, so you need not to have it on top. And maybe more descriptive name will be better like login_defs_to_obsolete, so if they again change keys, so can just add keys to this map and it will be translated to latest one.