diff options
author | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2010-09-03 19:01:30 +0200 |
---|---|---|
committer | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2010-09-03 19:02:42 +0200 |
commit | aaca9ce9927d9ea1568dfa89ba05a41b40333e9b (patch) | |
tree | 9a9122fca442186e7cc4ff5c95e2613959f7317c /lasso | |
parent | f245907b8ce8bc216d4f68eae215049dfde0ef2d (diff) | |
download | lasso-aaca9ce9927d9ea1568dfa89ba05a41b40333e9b.tar.gz lasso-aaca9ce9927d9ea1568dfa89ba05a41b40333e9b.tar.xz lasso-aaca9ce9927d9ea1568dfa89ba05a41b40333e9b.zip |
[ID-FFv1.2&SAMLv2] add more warning for failure to load metadata file
Report detailf of the failure through warning log.
Diffstat (limited to 'lasso')
-rw-r--r-- | lasso/id-ff/provider.c | 1 | ||||
-rw-r--r-- | lasso/saml-2.0/provider.c | 6 |
2 files changed, 7 insertions, 0 deletions
diff --git a/lasso/id-ff/provider.c b/lasso/id-ff/provider.c index 9b713fe4..43bfe90c 100644 --- a/lasso/id-ff/provider.c +++ b/lasso/id-ff/provider.c @@ -1019,6 +1019,7 @@ _lasso_provider_load_metadata_from_doc(LassoProvider *provider, xmlDoc *doc) g_return_val_if_fail(LASSO_IS_PROVIDER(provider), FALSE); if (doc == NULL) { + warning("Metadata is not an XML document"); return FALSE; } diff --git a/lasso/saml-2.0/provider.c b/lasso/saml-2.0/provider.c index 65bd579b..8f3807d9 100644 --- a/lasso/saml-2.0/provider.c +++ b/lasso/saml-2.0/provider.c @@ -396,6 +396,12 @@ lasso_saml20_provider_load_metadata(LassoProvider *provider, xmlNode *root_node) (! loaded_one_or_more_descriptor || (pdata->roles & provider->role) == 0)) { /* We must at least load one descriptor, and we must load a descriptor for our * assigned role or we fail. */ + if (! loaded_one_or_more_descriptor) { + warning("No descriptor was loaded, failing"); + } + if ((pdata->roles & provider->role) == 0) { + warning("Loaded roles and prescribed role does not intersect"); + } return FALSE; } |