I think we should add a parameter 'ConfigNode &node' to SyncSourceConfig.savePassword and so does PasswordConfigProperty.savePassword. what's your opinion? Another question is, since only PasswordConfigProperty knows what information it needs to store password, I'd like to add a virtual function 'getPasswordKey' and each child class implements it to get necessary information to the Key struct.