Blank reports from Kibana

System Details
Versions

Type
Version

Kibana
6.4.2

Elasticsearc
6.4.2

Server OS
Ubuntu

Browser
Chrome

Original Install Method

[ ] Release pkg link

Bug Details
I get blank images when I try to create reports of a Kibana Dashboard.
Bug Description
All other webpages that I've tried works just fine. Google, Aftonbladet.se and even Elastisearch on the same server as the Kibana-installation. I've tried to add a long delay, to let the page load properly. It seems that there is no delay. I've set the delay to 15s, but I get the mail within 15s from button pushed.
Sentinl Setup
sentinl:
  es:
    host: 'http://<IP_ADDRESS>'
    port: 9200
    protocol: 'http'
    results: 50
    timefield: 'doc.params.@timestamp'
    default_type: 'doc'
    alarm_index: 'watcher_alarms'
    alarm_type: 'sentinl-alarm'
  settings:
    email:
      active: true
      host: 'xxxx.xxxxxx.se'
      user: 'ebro'
      password: 'xxxxxxxx'
      port: 25
      ssl: false
    report:
      active: true
      engine: puppeteer
#      engine: horseman
      puppeteer:
        browser_path: '/usr/bin/google-chrome'
#      horseman:
#       browser_path: /home/adaadmin/Elastic/kibana/plugins/sentinl/phantomjs

Full Watcher JSON
{
  "actions": {
    "report_admin": {
      "throttle_period": "0h0m5s",
      "report": {
        "to"<EMAIL_ADDRESS>        "from"<EMAIL_ADDRESS>        "subject": "Rapport från Hultsfred",
        "priority": "high",
        "body": "Rapport över senaste timmen",
        "snapshot": {
          "res": "1920x1080",
          "url": "http://<IP_ADDRESS>:1337/goto/968963c7e3264e9a69b5164f98bf7f77",
          "params": {
            "delay": 5000
          },
          "type": "png",
          "name": "Rapport"
        }
      }
    }
  },
  "input": {
    "search": {
      "request": {
        "index": [],
        "body": {}
      }
    }
  },
  "condition": {
    "script": {
      "script": "payload.hits.total >= 0"
    }
  },
  "trigger": {
    "schedule": {
      "later": "every 1 hours"
    }
  },
  "disable": true,
  "report": true,
  "title": "Rapport Hultsfred",
  "save_payload": false,
  "spy": false,
  "impersonate": false
}

Screenshots (if relevant)

Errors in Browser Console (if relevant)

Kibana logs and/or server output (attached if relevant)
Any additional information

Missing execution right for PhantomJS and Puppeteer.


mini.map: allow setting height property of the map window

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.map
Description
It would be nice to be able to set the window height for the MiniMap, as currently the window is the full height of the Neovim window.
This causes the winblend to get applied to anything "behind" the map window, even when there is no part of the map on that line.

I have set my cursor line to white to demonstrate the effect.
It would be nice to be able to, for example set the height of the map window to the "text height" of its contents.
e.g.
window.height = vim.api.nvim_win_text_height(MiniMap.current.win_data[1], {}).all

Which currently has no effect when put in the config for mini.map, as the height is forced.
-- In function H.normalize_window_options
height = vim.o.lines - vim.o.cmdheight - (has_tabline and 1 or 0) - (has_statusline and 1 or 0),

See source@mini.map:1357-1382
As such, setting window.height currently seem to have no effect.

Thanks for the suggestion!
Setting window.height indeed has no effect because it is not a support configuration setting. Map window indeed is forced to full available height mostly to simplify code and available configuration. This is documented. Having more flexible window configuration is also documented as something that won't be done.
Closing as not planned.

Thanks for the quick response.
I completely understand not wanting to take on the additional complexity.
I guess I'll try monkey-patching it to see if I can make it work for my specific usecase.


Equidistant check fails for np.int32 coordinates

In GitLab by @Huite on Mar 11, 2019, 23:50
Interestingly, the TNO REGIS netCDF has RD new coordinates encoded as 32 bit integers. Unsurprisingly, this causes a failure in these lines:
https://gitlab.com/deltares/imod/imod-python/blob/master/imod/idf.py#L609
        if isinstance(dx, float) and isinstance(dy, float):
            ieq = True  # equidistant
            f.write(pack("?", not ieq))  # ieq

In GitLab by @Huite on Mar 13, 2019, 18:38
closed via commit c45f16c0d42e018ad9895821853c0e37db539c79


Use a real account for the demo experience

The demo experience is one route with some hardcoded API responses and "if demo" in a bunch of places.
Instead, the demo should be a real user and hit all the same code paths a normal user does.
It'll be a better demo experience, and it should enable external contributors without a device to more effectively contribute.

Would also be nice for the demo account to have

Ability to do snapshots, current demo experience "take snapshot" button is disabled
An option or just by default, make the demo account have about 5-10 trips so that we can see how certain designs affect the trip list. Currently demo account only has 1 trip by default

Here is a token to access a read only demo device in connect! Just place it under authorization in your local storage to try it out. You may need to enter an incognito session. This issue is now for moving our demo experience over to using this account/token. You should remove a bunch of if isDemo lines. @adeebshihadeh may want to make this a bounty
eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJleHAiOjE3NDg1ODI0NjUsIm5iZiI6MTcxNzA0NjQ2NSwiaWF0IjoxNzE3MDQ2NDY1LCJpZGVudGl0eSI6IjBkZWNkZGNmZGYyNDFhNjAifQ.g3khyJgOkNvZny6Vh579cuQj1HLLGSDeauZbfZri9jw


It is possible to activate "delegated harvesting" twice

0.6.28-BETA
Via helpdesk we got a report from a user:
"I accidentally clicked 'Activate Delegated Harvesting' twice, and now see two transactions for 6XEM pending!"
Of course this is not really a problem, because one of the transactions will be rejected and the user will not pay the fee twice, but still this is unpleasant. Is there a better way to avoid a "double activation"?

I can confirm that bug. Probably should be fixed in the gui.


THRIFT-406 Convert C++ Test to Boost::Test

This converts all C++ test to Boost::Test, integrates the DenseProtocolTest as a test like the others and removes C asserts from the TMemoryBufferTest.
Two test cases from DenseProtocolTest have to be commented, because they use the private vlqRead and vlqWrite Methods of apache::thrift::protocol::TDenseProtocol.

Merged in commit:
https://github.com/apache/thrift/commit/5ef662b9046d18aee224018061413ad7ee5d80a5


Indicate the full amount of data in the DCP, including managed access projects

The "headline" summary counts of Cells, Organs, Donors, Projects and Labs should include the contributions from managed access projects since this is non-PII and is not subject to filtering.

Obsolete.


Visualisation Userguide

This is a draft PR containing removed contents of the Visualisation section from the userguide.
They need completing prior to merging.

Re-add empty/template visualisation pages. This needs completing prior to re-merging.

Once complete this will close #49

This should now be complete. Most of the code included has been taken directly from examples or tested, but still possible there are small mistakes.

Thanks, have made that change. Also noticed I accidentally reverted to English spelling of Colour in a few places. Whereas I had intended to keep to US spelling to match the API.
Will merge after CI


[FEATURE] Use promtool to check syntax errors before the copy

Description
Currently, this role does not provide any mechanism to check if the Prometheus config files are correct. They can be checked in Molecule's verify with promtool, but this option does not provide enough information as the Prometheus service won't boot, failing the playbook in the converge stage.
It would be interesting to check those files with promtool before the copy.
Why is this needed?
In order to provide more descriptive information about bad configured alerts and etc
Additional Information
It would be interesting to try promruval

Included in 1.10.0 release


Thank you!

This page is very helpful!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2d80bc1b-b76b-e140-1cc4-e36f0968e2f1
Version Independent ID: 667ed2c3-cb70-9a07-d47c-8d49d4e20e4a
Content: Export restrictions on cryptography - UWP app developer | Microsoft Docs
Content Source: windows-apps-src/security/export-restrictions-on-cryptography.md
Service: unspecified
GitHub Login: @awkoren
Microsoft Alias: alkoren

Excellent!


Please Help NewsBlur experienced an error The error has been logged and will be fixed soon so you won't have to see this message again.

I am trying to selfhost newsblur from last 2 days & keep failing
now finally I see the page but with message
NewsBlur experienced an error
The error has been logged and will be fixed soon so you won't have to see this message again.
I checked other same issue & also tried make collectstatic
command etc
here is my log
[Aug 08 14:29:41]  ---> [Chrome] [0.01s] [AnonymousUser] Loading welcome
[Aug 08 14:29:41] Internal Server Error: /
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/django/core/handlers/exception.py", line 47, in inner
response = get_response(request)
File "/usr/local/lib/python3.9/site-packages/django/core/handlers/base.py", line 181, in _get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/usr/local/lib/python3.9/site-packages/sentry_sdk/integrations/django/views.py", line 67, in sentry_wrapped_callback
return callback(request, *args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/django/views/decorators/cache.py", line 44, in _wrapped_view_func
response = view_func(request, *args, **kwargs)
File "/srv/newsblur/utils/view_functions.py", line 32, in wrapper
return render(request, output[1], output[0])
File "/usr/local/lib/python3.9/site-packages/sentry_sdk/integrations/django/templates.py", line 95, in render
return real_render(request, template_name, context, *args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/django/shortcuts.py", line 19, in render
content = loader.render_to_string(template_name, context, request, using=using)
File "/usr/local/lib/python3.9/site-packages/django/template/loader.py", line 62, in render_to_string
return template.render(context, request)
File "/usr/local/lib/python3.9/site-packages/django/template/backends/django.py", line 61, in render
return self.template.render(context)
File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 170, in render
return self._render(context)
File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 162, in _render
return self.nodelist.render(context)
File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)
File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)
File "/usr/local/lib/python3.9/site-packages/django/template/loader_tags.py", line 150, in render
return compiled_parent._render(context)
File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 162, in _render
return self.nodelist.render(context)
File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)
File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)
File "/usr/local/lib/python3.9/site-packages/django/template/defaulttags.py", line 39, in render
output = self.nodelist.render(context)
File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)
File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)
File "/usr/local/lib/python3.9/site-packages/pipeline/templatetags/pipeline.py", line 136, in render
return self.render_compressed(package, package_name, 'css')
File "/usr/local/lib/python3.9/site-packages/pipeline/templatetags/pipeline.py", line 69, in render_compressed
return self.render_compressed_sources(package, package_name,
File "/usr/local/lib/python3.9/site-packages/pipeline/templatetags/pipeline.py", line 112, in render_compressed_sources
return method(package, paths, templates=templates)
File "/usr/local/lib/python3.9/site-packages/pipeline/templatetags/pipeline.py", line 148, in render_individual_css
tags = [self.render_css(package, path) for path in paths]
File "/usr/local/lib/python3.9/site-packages/pipeline/templatetags/pipeline.py", line 148, in
tags = [self.render_css(package, path) for path in paths]
File "/usr/local/lib/python3.9/site-packages/pipeline/templatetags/pipeline.py", line 143, in render_css
'url': mark_safe(staticfiles_storage.url(path))
File "/srv/newsblur/utils/pipeline_utils.py", line 14, in url
url = super().url(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/django/contrib/staticfiles/storage.py", line 147, in url
return self._url(self.stored_name, name, force)
File "/usr/local/lib/python3.9/site-packages/django/contrib/staticfiles/storage.py", line 126, in _url
hashed_name = hashed_name_func(*args)
File "/usr/local/lib/python3.9/site-packages/django/contrib/staticfiles/storage.py", line 417, in stored_name
raise ValueError("Missing staticfiles manifest entry for '%s'" % clean_name)
ValueError: Missing staticfiles manifest entry for 'css/reader/reader.css'
yesterday i tried hosting with other apps but failed brutally, today i am trying on totally new server but still failing
please help

after being depressed for few minutes, I tried make nb 1 more time & voila it worked
Thanks


Display a date/time on ads

to show when they were first seen (created_at).

wanna take a crack at this @jeremybmerrill ?

Yep
Jeremy B. Merrill
Sent from my mobile device
On Mar 13, 2018 1:57 PM, "Jeff Larson"<EMAIL_ADDRESS>wrote:

wanna take a crack at this @jeremybmerrill
https://github.com/jeremybmerrill ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/propublica/facebook-political-ads/issues/49#issuecomment-372760154,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAhdmqPcIzacNECf0qSqIvGJ8_xzozWyks5teAiHgaJpZM4Sm7X0
.


Downgrande homebridge from UI X

Describe Your Problem:
Just to confirm. If I click on the homebridge version and select an older version, it will be downgraded safely, right?
Screenshots:

Environment:

Node.js Version: 12.8.3
NPM Version: 6.14.8
Homebridge Version: 1.2.5
Homebridge Config UI X Version: 4.36.0
Operating System: Raspbian
Process Supervisor: hb-service

Yes.

Yes.

thank you for your quick response. 😅

thank you for your quick response. 😅


Pushkit completion handler

Make use of RNNotificationsStore for PushKit notifications and implement the new PushKit received handler withCompletionHandler.
It compiles and runs the ios e2e tests.

@yogevbd Thanks for merging. Seems that this also needed some change into the typescript side. I am working on this now, and I will do a new PR later today. Sorry


Can we integrate WinAppDriver with AzureDevOps?

Here is a link to the AzureDevOps guide for integrating WinAppDriver:


Add and export ConvexGeometry, ConvexBufferGeometry, and QuickHull

Needed for for https://github.com/kennethjiang/OctoPrint-Slicer/pull/101 .  These were taken from three.js examples and modified slightly to export properly.

it's merged and published to npm. You can now import them to Slicer plugin.


macOS: failing to retreive shell

(Sorry in advance if this is the wrong place to report this)
I installed macchina using Homebrew on macOS 12.1.
When I run it using this config, it fails to retreive the shell info:
interface = "en0"
long_uptime = true
long_shell = true
long_kernel = false
current_shell = true
physical_cores = true
show = [
	"Host",
	"Machine",
	"Uptime",
	"OperatingSystem",
	"LocalIP",
	"Shell",
	"Packages",
	"Resolution",
	"Terminal",
	"Processor",
	"ProcessorLoad",
	"Memory"
]

However, if I run it with macchina -c /dev/null it does print it. Screenshot below.
Not sure where the issue is but I guess maybe because the mod.rs module tries to look at /proc which doesn't exist on macOS? But not sure why it works with a null config...
(I use bash btw)

macchina will automatically pick up on this error (and report it) in the next release, so our work is finished :)

@grtcdr Amazing, thank you!

My pleasure :smile:


[e2e]: container workflow_dispatch main workflow_inputs slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/11098716015
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.workflow_dispatch.main.workflow_inputs.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.workflow_dispatch.main.workflow_inputs.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Mon Sep 30 03:18:39 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/11117969340
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.workflow_dispatch.main.workflow_inputs.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.workflow_dispatch.main.workflow_inputs.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Tue Oct  1 03:20:17 UTC 2024
Tests are passing now. Closing this issue.


Code Review 1

Fix the duplicated code:
https://github.com/LLumisS/Procedural-Map-Generation-Project/blob/b0a82a25a2b12bdcc88be9ba3b9b7f98efa5d684/server/middleware/checkRole.js#L5-L28
https://github.com/LLumisS/Procedural-Map-Generation-Project/blob/b0a82a25a2b12bdcc88be9ba3b9b7f98efa5d684/server/middleware/auth.js#L5-L23
Optimize request:
https://github.com/LLumisS/Procedural-Map-Generation-Project/blob/b0a82a25a2b12bdcc88be9ba3b9b7f98efa5d684/server/controllers/mapController.js#L124-L125
You can improve readability by using 'reduce':
https://github.com/LLumisS/Procedural-Map-Generation-Project/blob/b0a82a25a2b12bdcc88be9ba3b9b7f98efa5d684/server/controllers/mapController.js#L106-L112
Use 'beforeAll' instead:
https://github.com/LLumisS/Procedural-Map-Generation-Project/blob/b0a82a25a2b12bdcc88be9ba3b9b7f98efa5d684/server/tests/user.test.js#L26

Issues fixed.


Remove Duplicates

See here there are several comment descriptors. This is redundant and one of them is never used

In fact, there are many duplicates. This is probably a leftover from the previous themes that were used, which I may have forgotten to remove somehow when I was editing my settings.

Completed as of latest PR. Package and Upload pending!


scater_max

I encountered a very strange "illegle memory access" bug when I try to use the scater_max or softmax() which is based on the scater_max. I am sure that, the "source", "index", "dim_size" is right. Do anybody else meet the same prob?
File "/home/bl1/anaconda3/lib/python3.6/site-packages/torch_geometric/utils/softmax.py", line 21, in softmax
out = src - scatter_max(src, index, dim=0, dim_size=num_nodes)[0][index]
RuntimeError: CUDA error: an illegal memory access was encountered

Are you are sure that all your inputs are on the same device?

Are you are sure that all your inputs are on the same device?
Actually, I only use one GPU. The interest is that this error is related to my training iteration. I really can't understand it.

In addition, the scatter_mean is OK

Can you try scatter_max in a REPL or run the test suite of torch-scatter?

I comment the "out = src - scatter_max(src, index, dim=0, dim_size=num_nodes)[0][index]". It works fine now. There must be some werid thing related to scatter_max.  But I can't  figure it out now

Do you have multiple gpus? Which one do you use?

@VoVAllen Yes, I have multiple gpus, but I only use the second one. The werid thing is that scatter_mean is fine

I have an open issue regarding multi gpu support and PyTorch extensions: https://github.com/pytorch/extension-cpp/issues/18
I‘m gonna keep following this up.

You can use torch.cuda.set_device(1) to change the default context manager.

@VoVAllen I used the command of device = torch.device('cuda:1'). if they are the same?

No they are not. Custom cuda kernel will get CUDAStream from default device, not the device which tensor is on. You have to change the default device to let kernel get correct CUDAStream

Fixed now in torch-scatter-1.1.2 :)


Create a common ESLint config

Currently different versions of it are being copy-pasted across projects. Ideally it should be strict. Ideally there should be a JS version.

@Siegrift what do you think about keeping this sort of stuff in this repo as well?

The problem with this is that you want ESlint only as a dev dependency. If we create an eslint configuration here it will have other eslint plugins as dependencies. If you then install the commons, these dependencies will end up in prod deps.
One solution to this would be to have them as peer dependencies, but that is annoying to work with. I'd say that we can live with this for now and we can later convert it to monorepo (or separate package).


STUN and TURN server configuration

I have configured my TURN Server like the way you did but it seems like not working. Can I know how you guys made it? @muaz-khan

If you're using coTURN then please make sure that iptables has permission for outgoing and incoming traffic:
iptables -I INPUT 1 -i eth0 -p tcp --dport 3478 -j ACCEPT
iptables -I INPUT 1 -i eth0 -p udp --dport 3478 -j ACCEPT

If you're able to test your coturn using this demo:

https://webrtc.github.io/samples/src/content/peerconnection/trickle-ice/

and WebRTC fails instantly or doesn't connects then please make sure to open range of UDP ports: between 49152 and 65535.

I'm really confused with this COTURN server because I'm trying to run my application in windows and within the apache tomcat. But it seems like its only available in LINUX. @muaz-khan


Add openstack publishing promotion

Add Openstack publishing promotion

Partially pick https://github.com/projectcalico/calico/pull/7912.


Query - Dynamic/external rules, access different context information

HI @jruizgit,
I have the below questions:-

Is it possible to define the rules in an external file and then import them dynamically without restarting the service ?

Is it possible to access information from a different context ? In my case I am thinking to define multiple rulesets but I would also to co-relate the events that has been processed by the other rulesets. Also what is the maximum number of context information maintained ?

Cheers !!

Hi, for your first question I don't have an easy answer. You can start another process and load the rules in that process.
For #2, yes you can access information from a different context, for example:
with ruleset('a10'):
    @when_all(m.amount < c.s.ref_id('global').max_amount)
    def approved(c):
        print ('a10 approved {0}'.format(c.m.amount))

    @when_start
    def start(host):
        host.patch_state('a10', {'sid': 'global', 'max_amount': 300})
        host.post('a10', {'id': 1, 'sid': 1, 'amount': 400})
        host.post('a10', {'id': 2, 'sid': 1, 'amount': 900})

Changes in the referenced context will not be observed right away. Inside an action you can get another context state by calling: c.host.get_state(ruleset_name, id)

Thanks @jruizgit !!
Would appreciate if you could share a working piece of code in here or the documentation. Am not clear on the statement "inside an action you can get another context state by calling: c.host.get_state(ruleset_name, id)"

HI @jruizgit,
I have an array of rules and want to transfer it to durable's rule.
Here is my code in nodejs:
var rules = [{
    "id": "p0s0",
    "name": "",
    "action": {
        "url": "https://myhook.com",
        "id": "req-url",
        "method": "POST"
    },
    "type": "value",
    "condition": {
        "variable": "PH",
        "properties": {
            "type": "<",
            "value": 10.0
        }
    }
}];
for (var i = 0; i < rules.length; i++) {
    var rule = rules[i];
    var name = rule.id;
    var prop = rule.condition.variable;
    var value = rule.condition.properties.value;
    var fn = function () {
        whenAll: m[prop] < value
        // consequent
        run: console.log('do something ')
    };

    d.ruleset(rule.id, fn);
}```
How can i do it? please show me the way!
Thank!

Hi, it might be easier for you to use the engine directly.
var d = require('durableEngine');
var h = d.host();
h.registerRulesets(null, rules);
var a = d.application(h);
a.run();

Your JSON rules will need to follow the following format:
[{
    name: {
        all: [
            {first: {t: 'deposit'}},
            {second$not: {$and: [{t: 'withrawal'}, {ip: {first: 'ip'}}]}},
            {third: {$and: [{t: 'chargeback'}, {ip: {first: 'ip' }}]}}
        ],
        run: function...
    }
}]

Would you mind opening a separate issue to keep track of your question? I haven't fully documented the JSON format.

Thank  @jruizgit!
I have a new approach to do this and it worked as expected.
I'll write a separate issuenext time.

@chinhls can you please share your approach?
Many thanks!

@jruizgit
If I'm passing rules via JSON, then how can we define a function for run within the JSON rule like as you said above.


Add support for ExternalName services

Related issue: #861

@njuettner I added docs for using External DNS with ExternalName services. Please let me know if there is anything else needed before the merge.

Sorry, one minor change request, could you format this line: source/service.go:407: File is not goimports-ed (goimports)

@njuettner Sure, I fixed the indentation. I think spec failure is not related to the change, right?

/lgtm
/approve


[TC-377] Default profiles for EDGE and MID are missing after initial install

I suggest we don't add these to the seeds, but create a "profile exchange" that people can dl the right profile from.
I'll take a first stab at this.
Author: Jan van Doorn
JIRA Link: https://issues.apache.org/jira/browse/TC-377

@limited default profiles have been provided on our website under downloads. http://trafficcontrol.apache.org/downloads/profiles/2.0.x/default/

Closing, we are providing links above.


Running git commit --> Could not decode JSON respose

This is my test case
I made some minor changes ( for testing purpose )
I run git add .

Error: Could not decode JSON respose ( a typo )
Is there a verbose mode, so that the user will know what the error is and act accordingly ?
Thanks

Similar --
❯ git commit -a
🤖 Asking GPT-3 to summarize diffs...
Error: Could not decode JSON respose

Good point and thank you both for the feedback. This should be fixed in #15 giving us a better error message into what is going on.
I'll ship a new version 0.1.3 with this fix

Thanks @zurawiki , I updated the package, and now I see the error message, seems like it is an issue in gptcommit
{"error":{"code":null,"message":"Invalid URL (POST /v1/completions1)","param":null,"type":"invalid_request_error"}}

I opened a PR regarding to the issue above - https://github.com/zurawiki/gptcommit/pull/16

Thanks @zurawiki , I updated the package, and now I see the error message, seems like you have a typo, it should be completions instead of completions1.
{"error":{"code":null,"message":"Invalid URL (POST /v1/completions1)","param":null,"type":"invalid_request_error"}}

Thanks for pointing that out! I was rushed last night uploading v0.1.3 and left that wrong endpoint in as a way to debug, sorry!
The fix is already landed to master and I've published v0.1.4. I'm still curious as to what the original error you were seeing was.

@zurawiki It's all good now.  The message comes back with reason of "insufficient quota".  Now user will have insight of what's going on!

Closing this ticket as resolved.


Pre-fetch cobolcheck

@axtens There was code to use a pre-fetched cobolcheck in each exercise's test.sh file, but CI never downloaded it. This PR does that, and also applies the same logic to the powershell version. It did mean that I needed to add that extra "check if the file is already there" check to each exercise's test.ps1 script.

Thank you!


Small fixes to the examples

supervisor would throw a nil dereference error on startup
server html was referencing a nonexistent protobuf attribute

@UVduane thank you for the PR.
Please sign the CLA so that we can accept contributions.

This also needs a more detailed description of the problem, steps to reproduce, etc.

@tigrannajaryan Thanks.  I'm working through the CLA process with company right now.  Process-wise, would you like issues entered for the problems I found?  (I know some projects prefer this, and I probaly should have asked first)

@tigrannajaryan Thanks. I'm working through the CLA process with company right now. Process-wise, would you like issues entered for the problems I found? (I know some projects prefer this, and I probaly should have asked first)

We prefer that too, although for small issues it is not mandatory.
However, I would definitely prefer that this PR is split into 2 separate PRs, each fixing one of the problems, since they are not related in any way. And, importantly, for bugs we need reproduction steps to confirm the bug.

You got it.  I'll close this and do some housekeeping w/ issues and PRs to split it all up and document everything.  Thank you for getting the noob pointed in the right direction.


add next steps to service entity

Change description
Shows next steps

If there's no ladder, it remains the same and shows rank

Type of change

[ ] Bug fix (fixes an issue)
[ ] New feature (adds functionality)
[ ] Improvement (improves codebase without changing functionality)

Related issues

Fix #1

Checklists
Development

[ ] Lint rules pass locally
[ ] Application changes have been tested thoroughly
[ ] Automated tests covering modified code pass

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Code review

[ ] Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as necessary
[ ] "Ready for review" label attached and reviewers assigned
[ ] Changes have been reviewed by at least one other contributor
[ ] Pull request linked to task tracker where applicable

minor nit, can we add hr or some kind of divider between current level, next level, and what's next to create a similar visual separation like in the Cortex ui


Turn Teams on or off for your entire organization

It's stated that this is no longer an option for any tenant other than for ones with EDU licensing SKUs. But it's not, it's still available for Worldwide licensing SKU's We recently (needlessly) opened a Microsoft Premier case for an error "You're missing out ask your admin to enable Microsoft teams for <orgname>" where we omited this as a troubleshooting step due to this article and the fix was to revert to the old admin center per the instructions for EDU licensing.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a5473bf0-ba2d-6c42-d57d-b61af64452d3
Version Independent ID: 7195478d-3ef4-be8c-2920-3fdba3e70c7e
Content: Turn on Microsoft Teams in your Office 365 organization - Microsoft Teams
Content Source: Teams/Office-365-set-up.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@Ganym3de  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@Ganym3de thank you for reaching out and helping improve MS Docs. Are you suggesting removing the note at the bottom of the article about tenant-level control?

@Ganym3de We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

You do not need the original poster's interaction in order to fix this.  The original message is sufficient.  This line needs expanding to include Worldwide Sku: "
The exception to this change is Microsoft Teams for Education (EDU)"


fix: shabbos yuntif does not have a parsha

Fixes #16.
Parsha will be returned as null if there is no parsha that week (the case of yuntif).

I have 2 comments:
Now, it looks like you can never have Vezos Habracha as a Parsha, but it should probably show up as a Parsha for Simchas Torah
You added a new function comingShabbos, I'm leaving a comment there

Right, we don't have Vezos Habracha as a parsha anymore, that makes sense to me.
Can you post the comment? I don't see it yet

Sorry, I thought it was submitted. I just submitted now


Replace reactor-core deprecated ReplayProcessor in EH ManagementChannel with Sinks, minor renames to remove use of the term processor*

API change check
API changes are not detected in this pull request.


Fix possible cross-cluster connection drops on agents restart when clustermesh is enabled

Backport of https://github.com/cilium/cilium/pull/27575 for v1.14. Currently used to build the images for testing https://github.com/cilium/cilium/pull/27232.

/test-backport-1.14

/test-backport-1.14

/test-backport-1.14


[data-plane] refresh communication-job-router-rest sdk

https://github.com/Azure/azure-sdk-for-js/issues/30680

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure-rest/communication-job-router

need use main refresh


support python 3+ on learn_embedding method.

learn_embeddings needs minor modification to accept the python3+ 's map function. implemented below
def learn_embeddings(walks):
'''
Learn embeddings by optimizing the Skipgram objective using SGD.
'''
#print(type(walks))
#walks = [map(str, walk) for walk in walks]
walks= [str(j) for i in walks for j in i]

Actually that alone does not fix the issue with Python3 implementation. Above solution I offered has to be changed to the following.
def learn_embeddings(walks):
'''
Learn embeddings by optimizing the Skipgram objective using SGD.
'''
walks = [str(walk) for walk in walks]
model = Word2Vec([walks], size=args.dimensions, window=args.window_size, min_count=1, sg=1, workers=args.workers, iter=args.iter)
model.wv.save_word2vec_format(args.output)

print('vocalublary', list(model.wv.vocab), 'length', len(list(model.wv.vocab)))

return

why you add '[]' for variable 'walks' again? this mean get walk list embedding instead of node in walks?

fix at #35

hi, thanks your modification! But in my try, walks = [str(walk) for walk in walks] doesn't work, and the final embeddings are of random walks rather than nodes. In my opinion, only list() is needed. Maybe, you can try this walks = [list(map(str, walk)) for walk in walks]. Meanwhile, '[]' can be removed.

in main.py
function learn_embeddings(walks)
can change
'walks = [map(str, walk) for walk in walks]' to 'walks = [[str(j) for j in i] for i in walks]'
to reach the requirement of Python3

can change this sentence to
walks = np.array(walks, dtype=str).tolist()
the origin map function aims to change the type of each element into str

The following works fine on python3:

def learn_embeddings(walks):
	'''
	Learn embeddings by optimizing the Skipgram objective using SGD.
	'''

	walks = [list(map(str, walk)) for walk in walks]
	model = Word2Vec(walks, size=args.dimensions, window=args.window_size, min_count=0, sg=1, workers=args.workers, iter=args.iter)
	model.wv.save_word2vec_format(args.output)

	print('vocalublary', list(model.wv.vocab), 'length', len(list(model.wv.vocab)))

	return


Cheese-UserJ

This wonderful little project makes delightful little (non-edible) cheeses.
I really wanted to focus on getting the perspective right, which is why the camera control got in there. I've always been inspired by 3d rendering toy projects, and while this isn't even close to that kind of stuff, thinking in 3d still twisted my brain in a way that forced me to step back and try again quite a few times.

Nice work! It's recognizable as cheese and you used some great techniques to get the 3D effect.
 You can order it here.
Some feedback to keep improving this. The holes are distinctly 2D, how could we give them depth? And when the cheese wheel occludes itself the perspective breaks (large cheese angles).


Problem when deserialize Request body - Testing

I am facing some JSON decoder problems. When write unittest for wsgi.py, it raise this exception 'ValueError: No JSON object could be decoded'. Go deep in there, i use simple,json module for better error msg, then i get bellow msg:
simplejson.scanner.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

In most cases json.loads or simplejson.loads - JSONDecodeError: Expecting value: line 1 column 1 (char 0) error is due to :
non-JSON conforming quoting
XML/HTML output (that is, a string starting with <), or
incompatible character encoding

Ultimately the error tells that at the very first position the string already doesn't conform to JSON. I recheck this, but still dont know reason and solution:
self.body = '{"cloud":"cloud1"}'

I try to print body before json.loads(), it return exactly {"cloud": "cloud1"} and when run json.loads('{"cloud":"cloud1"}'), it works like a charm. But, if json.loads(body) it doesnt work.
P/s: falcon.Request doesnt have body attribute. It make me confused at the first time, but now i understand why:
It’s called stream instead of body to emphasize the fact that you are really reading from an input stream; Falcon never spools or decodes request data, instead giving you direct access to the incoming binary stream provided by the WSGI server.

P/s2: I just push to my individual repo, for test any solution without affect original CAL repo. Check test_wsgi here

@ntk148v will take a look and give feedback later.

Your code base is not compatible with python 3, please use json.loads(datastring.decode()..) instead of json.loads(datastring,..)
Your TestMiddleWare never call your two hooks, so this will always lead to JsonMalform error.

@hieulq

I try decode request body before put it in json.loads() method. It still get error, so i remove it.
I pretty sure this is not the reason. Because process_request middleware raise an error(ValueError with JSON), falcon will skip any lower layers/middlewares, in this case is process_resource. So, 2 hooks never called.

I still think problem is body of request. It lead to ValueError, like i said.

@ntk148v IMO, falcon itself will call to the stack of middleware function (process_request) before processing resources (with hooks), the problem here is in this stack context, there is only one request object, or you can call a stream, right?
So, in your test case, there are 2 middleware based on 2 hooks. When you simulate request in falcon, the request will be sent to the 1st middleware (_first_hook), grab the stream body. Then next to the 2nd middleware with the empty body (because the 1st middleware have eaten whole of stream body), so the body is always empty and make falcon raise ValueError.
References is falcon flow with middle, ref: http://falcon.readthedocs.io/en/stable/api/middleware.html?highlight=middleware
mob1.process_request     mob2.process_request         mob3.process_request             mob1.process_resource                 mob2.process_resource                     mob3.process_resource             <route to responder method>         mob3.process_response     mob2.process_response mob1.process_response
I think we are using middleware with hooks in falcon in wrong way. Because the old wsgi code is custom wsgi, but falcon use middleware in very different way.

@hieulq it may be right, i will recheck this then tell u soon.

@ntk148v take a look at this example:
http://falcon.readthedocs.io/en/stable/user/quickstart.html#more-features
You can see they create 3 different middlewares.

@ntk148v I prefer you rewrite the test_wsgi structure for handling right behaviors of falcon. The 2 hooks need to be put in validator context of request.
Another problem here is currently we are missing test case for process resources, right?

@hieulq i read this tut, but i think using 1 middleware still works. (and reality, it doesn't, too bad). I will rewrite it.

@ntk148v +2, do we need to delete the unittest branch?

@ntk148v ack, too fast too furious
Thanks for your effort. Keep moving forward for all tasks of this week.


[QUESTION] While installing TTS, it is crashed.

Hello,
After cloning the latest audio-webui git repo, run ./run.sh file on ubuntu22.04 (python3.10, GPU)
But while installing TTS, it is crashed.
Exit code: 1
Please help me resolve this problem.

I will likely be moving TTS to an extension so it can be disabled whenever such an issue occurs, as i don't want users to be stuck because of a feature they weren't even planning to use.

TTS has been moved to an extension, if you want to use audio-webui without TTS, you'll need to use the TTS-extension branch. Which has the TTS features removed so they can be loaded through an extension instead.


Padding in forms

Add some padding in forms. The input fields run out of the screen on the right when viewing on mobile.

hey @kunal-kushwaha  is this issue still open ?

hi, can I work on this?


Downloading files from a bucket

Is it possible to download files from buckets? In python, you're able to use download_file() (https://ibm.github.io/ibm-cos-sdk-python/reference/services/s3.html#S3.Bucket.download_file), but it seems this is missing in JS.

@justin-chen I'm looking into this now.

@justin-chen JS doesn't have download_file(). I'd suggest you use createWriteStream with getObject(). A simple example of this can be found below:
var params = { Bucket: ‘my-bucket-pc’, Key: ‘foo’ }; var file = fs.createWriteStream(‘/Users/me/Downloads/foo’); function doGetObject() {    console.log(‘Getting object’);    cos.getObject(params).createReadStream().pipe(file); }


Fix vaccination history

RKI changed headers
This fixes parts of issue #237

@marlon360 i know that you are very busy at the time, but please approve and merge this PR a.s.a.p. because /vaccination/history do not work again (since yesterday!) and hopefully you find some time to answer my question in #237

If you @marlon360 decide to include the latest history entry in the coverage endpoint (I'd find that very useful) you may merge #239 instead.

Closed, project accepted #239


issue-manager index.html

In the index.html file when displaying information about the github issue its reference as ${issue.githubIssue.url} when running this the following error occurs.
Exception evaluating SpringEL expression: "issue.githubIssue.url" (index:16)
The expression needs to be changed to ${issue.gitHubIssue.url}  to display the data on the page.

I wonder if that changed at some point. I did test every line of this code. But the project is still on snapshots, two years later.

I've completed the second chapter and that is the only issue i've found regarding that project

Good! I hope you're enjoying it.


Create new release

Hello crazycodeboy,
I've made a pull request to your nice module. But when I'm getting the package with NPM my new feature isn't in it. Could you create a new release?
Thanks!

Released, thanks for your contribution.


Deriving Clone on ObjectSubclass is unsound

It would create a clone of just the instance private struct. Calling ObjectSubclass::get_instance() on it would then get a reference to some random memory.
This needs to be prevented somehow. Maybe via a cleverly placed assert_not_impl_all in the glib_object_subclass! macro.

So really anything implementing ObjectSubclass must not be constructed without going through some glib functionality? That seems like it would be impossible to enforce, since Clone is just one of an endless amount of ways of instantiating a type.

On 16 October 2020 00:33:30 EEST, Jonas Platte<EMAIL_ADDRESS>wrote:

So really anything implementing ObjectSubclass must not be constructed without going through some glib functionality? That seems like it would be impossible to enforce, since Clone is just one of an endless amount of ways of instantiating a type.

We could require a non cloneable ZST to be stored inside the struct, and make that only constructable inside glib.
All not beautiful
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Okay, so I am right? Ouch 😅
I guess a good solution API-wise would be having a custom method receiver wrapper type (still unstable unfortunately).
Could a wrapper type maybe work without it being the receiver in an impl MySubclass block, so users have to instead write impl Obj<MySubclass> / impl Whatever for Obj<MySubclass>?

That could also work indeed, so we already have two potential solutions :) I'd prefer yours if custom receiver types were stable but like this both are a bit suboptimal. Will think about it a bit more :)


Generalized string literals (bytes and raw sequences)

Accepts string literal of the form b"xxx" and r##"\n\tblabla"## as described in the rust reference.

I was just about to open another issue on this subject when unnoticed this. It would be great to be able to have a built in way of matching rustlike string, or at least expressing escape sequences so that we can construct our own.

Yes, it would be cool! I'm wondering if we need to modify something else than the parser.

Triage: string literals are now automatically supported (thanks to the parser using syn).

Triage: string literals are now automatically supported (thanks to the parser using syn).


Update Ionic 2 example to RC.0

Current behavior
Readme explains bootstrapping for Ionic Beta
Expected/desired behavior
Readme explains bootstrapping for Ionic RC
Edit: Trying to put TranslateModule.forRoot() into imports in app.module.ts will kill the ngc.

Yes, it's definitely on my todo list :)

Any quick tips on how to get it to work with Ionic RC.0?

I suppose it should work just like the regular documentation for non ionic apps, just import the module and call forRoot() (with or without parameters, depending on what you need, check for the readme for that)

Unfortunately it doesn't work. I get
ngc error: Error: Error encountered resolving symbol values statically. Function calls are not supported. Consider replacing the function or lambda with a reference to an exported function, resolving symbol AppModule in xxx/.tmp/app/app.module.ts, resolving symbol AppModule in xxx/.tmp/app/app.module.ts
Using
imports: [         IonicModule.forRoot(MyApp),         HttpModule,         TranslateModule.forRoot() ],

Hi! That's #218 .

Ah, so it is. Thank you.

On my side it writes:
[16:58:58]  Error: Module /node_modules/ng2-translate/ng2-translate.js does not export TranslateLoader (imported by /.tmp/app/app.module.js)

TranslateLoader and TranslateStaticLoader can be imported from 'ng2-translate/src/translate.service', but calling TranslateModule.forRoot() will cause the build process to fail.

dev mode works fine if we include {TranslateLoader , TranslateStaticLoader, TranslateService } from 'ng2-translate/src/translate.service',
"ionic build" fails
ngc error: Error: Error encountered resolving symbol values statically. Function calls are not supported. Consider replacing the function or lambda with a reference to an exported function (position 30:23 in the original .ts file), resolving symbol AppModule in C:/Users/dsc/dev/23_angular__rhenus__information/.tmp/app/app.module.ts
at simplifyInContext (C:\Users\dsc\dev\23_angular__rhenus__information\node_modules@angular\compiler-cli\src\static_reflector.js:469:23)
at StaticReflector.simplify (C:\Users\dsc\dev\23_angular__rhenus__information\node_modules@angular\compiler-cli\src\static_reflector.js:472:22)
at StaticReflector.annotations (C:\Users\dsc\dev\23_angular__rhenus__information\node_modules@angular\compiler-cli\src\static_reflector.js:61:36)
at _loop_1 (C:\Users\dsc\dev\23_angular__rhenus__information\node_modules@angular\compiler-cli\src\codegen.js:53:54)
at CodeGenerator.readFileMetadata (C:\Users\dsc\dev\23_angular__rhenus__information\node_modules@angular\compiler-cli\src\codegen.js:66:13)
at C:\Users\dsc\dev\23_angular__rhenus__information\node_modules@angular\compiler-cli\src\codegen.js:100:74
at Array.map (native)
at CodeGenerator.codegen (C:\Users\dsc\dev\23_angular__rhenus__information\node_modules@angular\compiler-cli\src\codegen.js:100:35)
at codegen (C:\Users\dsc\dev\23_angular__rhenus__information\node_modules@angular\compiler-cli\src\main.js:7:81)
at Object.main (C:\Users\dsc\dev\23_angular__rhenus__information\node_modules@angular\tsc-wrapped\src\main.js:30:16)

Same for me.
Error: Module node_modules/ng2-translate/ng2-translate.js does not export TranslateLoader (imported by /home/SideMenu/.tmp/app/app.module.js)

https://github.com/ocombe/ng2-translate/issues/218 has some sort of solution

Thanks for the quick fix!

Be careful, for ionic the configuration should be :
useFactory: (http: Http) => new TranslateStaticLoader(http, 'assets/i18n', '.json'),
instead of
useFactory: (http: Http) => new TranslateStaticLoader(http, '/assets/i18n', '.json'),
(mind the slash before assets)


Fix a unit test assertion

@ybettan @yevgeny-shnaidman What's the purpose of this assertion?
Expect(res).To(Equal(res))

Originally posted by @erusso7 in https://github.com/qbarrand/oot-operator/pull/69#discussion_r886883341

@ybettan Sorry for the confusion. The text of the issue comes straight from @erusso7's comment. I created this issue as a follow-up to fix the assertion.

kubectl apply -k https://github.com/qbarrand/oot-operator/config/default

https://github.com/qbarrand/oot-operator/config/default


Gradient background doesn't render in production build

I am trying to fill a line chart with a gradient background. It works when I run it locally but doesn't work in a production build.
Expected Behavior
Should render the gradient when passed as backgroundColor. [It works in development environment]

Current Behavior
BackgroundColor is transparent.

Steps to Reproduce
Data:
{
  data: chartData,  // this is an array of point values
  label: 'Metrics',
  fill: true,
  backgroundColor: (dataset: DatasetController<'line'>) => {
    const ctx = dataset.chart.ctx;

    const gradient = ctx.createLinearGradient(0, 0, 0, 224);
    gradient.addColorStop(0, 'rgba(8,164,225,1)');
    gradient.addColorStop(1, 'rgba(255,255,255,0)');

    return gradient;
  }
}

Context
I noticed while debugging that the CanvasGradient is correctly passed to chartJS even after which its not rendered. Is there any component specific that I need to register manually in v3 ?
Environment

Angular v10
ng2-charts v3.0.0-rc.3
Chart.js v3.3.2
Browser name and version: Google Chrome 91.0.4472.114

I'd guess its a tree shaking issue, did you import and register the filler plugin?

Thanks @kurkle Works perfect once I register the Filler. Do we have this documented somewhere? It would be good to have a list of components that we need to register for certain functionalities.

The things that are available are listed there: https://www.chartjs.org/docs/latest/getting-started/integration.html#bundlers-webpack-rollup-etc
There is a note about being implemented by the filler plugin at the area charts: https://www.chartjs.org/docs/latest/charts/area.html


Fix of mismatch of data and control plane blocking istio proxy reset

Mismatch between data and control planes should not block Istio reconcilation if control plane is consistent
Related issue: https://github.com/kyma-incubator/reconciler/issues/1053

/retest

/retest


Java solution for Ex1

Here is a Java solution for exercise 1.
AFAIK it is correct. Please check before merging that the numbers found for the two unknown cases in the testcases are correct.

@danielleberre, numbers look good! Either way, second exercise is about double-checking these travel plans with the external tracer tool :)


Fixed build for Alpine Linux

To make mavkink-router build on minimalistic Alpine linux distribution I had to:

replace strndupa with strndup. stdndupa uses alloca and is not portable. Alloca() allocates memory within the current function's stack frame, so when replacing with strndup we have to call free manually. There were some places where alloca was incorrectly used
If TCSETS2 or TCGETS2 are not defined then use TCSETS and TCGETS

replace strndupa with strndup

We already have our own definition of strndupa of it's not defined. Why is it not working for you?

TCSETS2 or TCGETS2 are not defined then use TCSETS and TCGETS

This breaks cases in which we want rounded nauseated like 500k / 1.5M. TC[GS]ET doesn't work with those.

Sorry - I see that strndupa problem was solved 2 months ago and I didn't use the latest from master.
I will close this.


Add Python 3.12 Beta to CI

Looks like jq does not currently have python 3.12 compatibility. We'll need to take a look at that.
Closes #19

Looks like this branch needs to be updated.  Let me know if you want a review.


Corrupting video when trimming

Trimming is working perfectly for a video which is recorded with out audio. But when i trimmed(trying to get last 10sec video from recorded video) its adding blank frames at the end for 2 sec. Any help here?

@nareshy-menlo can you please share a code with me to test it?


Unable to start - listening on port 8085

Problem/Motivation

Tomcat will not start

Expected behavior

Tomcat fails to start

Actual behavior

Failed to start

***************************
APPLICATION FAILED TO START
***************************
Description:
The Tomcat connector configured to listen on port 8085 failed to start. The port may already be in use or the connector may be misconfigured.
Action:
Verify the connector's configuration, identify and stop any process that's listening on port 8085, or configure this application to listen on another port.
2021-07-21 11:26:54.339  WARN 349 --- [           main] o.s.b.f.support.DisposableBeanAdapter    : Invocation of destroy method failed on bean with name 'database': java.lang.NullPointerException
[cmd] /usr/bin/unimus exited 1
[cont-finish.d] executing container finish scripts...
[cont-finish.d] 99-message.sh: executing...

Steps to reproduce

Start Unimus

Proposed changes

Not sure

Shush up bot - I'm just busy ;)


Seeing Unknown for Contacts

I have an issues where the first and the last name does not appear in the search and there by not creating the file with the desired name.

It does not happen for my user id though, I configured the right API access as well. Iam using this for our work domain
Any idea why that would be the case.

I've seen this before.  I suspect it is the External Directory Sharing domain setting which would limit fields that are accessible via the API.
I don't know if you are an admin on your domain or if you're able to validate that.

I have the same problem. This is the setting:

Should the first option be selected for this to work instead?

I believe so.  If you are able to try it to confirm, that would be great.  Note that changes like this on the google side might take a while to take effect, so you might have to give it some time before trying.

Hm, looks like this made it worse. Directory doesn't yield any results and in the obsidian log this is what I now see:
unable to query directory: The G Suite domain admin has disabled external directory sharing. See more details at https://support.google.com/a/answer/6343701

Update: waiting didn't help. WIth the Public data and authenticated user basic profile fields no directory results are returned at all. ☹

bummer :/ thanks for checking.  it has to be some google setting; i'll dig around some more and see if i can figure it out

I believe this could be related to https://issuetracker.google.com/issues/196235775?pli=1

it seems google finally fixed this according to the status here: https://issuetracker.google.com/issues/196235775
hopefully this resolved your issue


sbbr-test: define ACS_VERSION for RELEASE

Building with UEFI_BUILD_MODE=RELEASE in build-sct.sh raised the following error:
/home/user/workspace/arm-systemready/SR/scripts/edk2-test/uefi-sct/SctPkg/TestInfrastructure/SCT/Drivers/StandardTest/StandardTest.c: In function ‘StslBeginLogging’:
/home/user/workspace/arm-systemready/SR/scripts/edk2-test/uefi-sct/SctPkg/TestInfrastructure/SCT/Drivers/StandardTest/StandardTest.c:1092:75: error: expected ‘)’ before ‘ACS_VERSION’
     SctSPrint (Buffer, EFI_MAX_PRINT_BUFFER, L"Arm ACS Version: %s\n", L""ACS_VERSION);
                                                                           ^~~~~~~~~~~

edk2-test-bbr.patch adds a print using the macro when applied to uefi-sct, so the macro has to be defined in both cases.

Hi @eric-ch ,
Thank you for the contribution. Will merge the PR.
SystemReady ACS Team


Bug - No products showing up for service

I met with a service partner today that is working on data plane and management plane Private Preview for a new service and initial offering.  We walked through what they needed to do.  Then I provided a quick demo of the release planner Beta 2 (PROD) and Beta 3 (PPE).  They wanted to create a release plan for tracking Private Preview but the product list failed to load for the service.  I will put the screenshot and details in the engagement experience eng teams channel.

This is related to https://github.com/Azure/azure-sdk-tools/issues/5180 which has been fixed.
@ladonnaq if you still see issues, let me know


async env variables

Hey,
I was wondering if it was possible to update env variables async at boottime? I have an env variable based on a server-side call, so would love to have something like:
runtimeEnvs: {
    MY_ENV: async () => { return await fetch("myurl.com") }
}

It's possible right now, but not the way you expected.
import { createEnv as createT3Env } from "@t3-oss/env-core";
import { z } from "zod";

async function loadEnv() {
  const value = await fetch("myurl.com").then(async (r) => await r.json());

  return createEnv({
    server: {
      MY_ENV: z.string(),
    },
    runtimeEnv: {
      MY_ENV: value,
    },
  });
}

// Usage
const env = await loadEnv()

Did it work for you @SaadBazaz ?


Add performance benchmarking

Only when logLevel === LOG_LEVELS.log

Add ?debug-cp to the url to enable log level at start


Samsung Music Frame getting Error communicating with socket, resetting connection

What version of Music Assistant has the issue?
2.0.7
What version of the Home Assistant Integration have you got installed?
2024.6.2
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
The music will stop after a few seconds of playing, this only happens on the Samsung Music Frame
How to reproduce
Play any music from any source to the player will received the error and music stop playing
Music Providers
Youtube, Spotify
Player Providers
Samsung Music Frame
Full log output
Logger: pychromecast.socket_client
Source: /usr/local/lib/python3.12/site-packages/pychromecast/socket_client.py:412
First occurred: 11:59:05 AM (2 occurrences)
Last logged: 8:32:39 PM
[Music Frame(<IP_ADDRESS>):8009] Failed to connect to service HostServiceInfo(host='<IP_ADDRESS>', port=8009), retrying in 5.0s
[Music Frame Study Room(<IP_ADDRESS>):8009] Failed to connect to service HostServiceInfo(host='<IP_ADDRESS>', port=8009), retrying in 5.0s
Logger: pychromecast.socket_client
Source: /usr/local/lib/python3.12/site-packages/pychromecast/socket_client.py:655
First occurred: 8:32:39 PM (1 occurrences)
Last logged: 8:32:39 PM
[Music Frame Study Room(<IP_ADDRESS>):8009] Error communicating with socket, resetting connection
Additional information
No response
What version of Home Assistant Core are your running
2024.7.1
What type of installation are you running?
Home Assistant Core
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

What players do you have that do work?

B&O M3 is working without issue

If it isn't a networking issue then might be a Samsung problem. They are known to have poorly implemented protocols. Please attach  (drag and drop) the FULL log. This will be looked at in due course

Ok sure, I will do it after work. Thank you very much


Multi videos picker

I didnt upload multiple videos please update it for also multi videos
Uint8List? videoData = await ImagePickerWeb.getVideoAsBytes();
Thank you so much for amazing packge

Hi, @awaisdev5765 I'm going to take a look at this.


Multipart messages

Multipart messages give an error message:
    to_encode = MIMEText(x)
  File "/usr/local/Cellar/python/3.7.0/Frameworks/Python.framework/Versions/3.7/lib/python3.7/email/mime/text.py", line 34, in __init__
    _text.encode('us-ascii')
AttributeError: 'list' object has no attribute 'encode'

I think the code currently assumes a single part, that can be converted into text (and base64 encoded so it fits the block cipher). Is there a more sophisticated way to 'flatten' a multipart message ready for encryption?
This would help towards supporting Content-Type multipart/signed also.
I think I found how Thunderbird is doing it, but I'm not good at reading Perl: https://dxr.mozilla.org/mozilla-esr45/source/security/nss/cmd/smimetools/smime
I'm happy to try and work on this, but if you have some hints on how to start that would be great.

Thank you.


@Null*arked on record components

See summary in https://github.com/jspecify/jspecify/issues/43#issuecomment-881061588, with discussion starting in https://github.com/jspecify/jspecify/issues/43#issuecomment-700277489.
It might be that all we can do is to encourage tools to report an error if compiling code that uses @NullMarked/@NullUnmarked on a record component. And probably people won't even think to try that in the first place, so we're probably fine in practice no matter what.

I'll mention in a NullMarked/NullUnmarked javadoc update that they have no meaning on a record component.

Fixed by #349


RBD: Advertise health conditions cap

Advertise the volume condition cap for RBD and set the volume condition for the missing block mode.

@mergifyio queue


Signed HTTP Exchanges?

Is there possibillity to support the Signed HTTP Exchanges?
Example here:
https://wicg.github.io/webpackage/draft-yasskin-http-origin-signed-responses.html
How can we implement this on our server?
I am running Nginx.
Or, is there possibillity with CloudFlare?
https://www.cloudflare.com/website-optimization/ampersand/
and
https://blog.cloudflare.com/real-urls-for-amp-cached-content-using-cloudflare-workers/
Thanks!

https://ampbyexample.com already supports Signed HTTP Exchanges (well, until it broke...) it was added in  https://github.com/ampproject/amp-by-example/pull/1592. It should be possible to achieve the same outcome via the CloudFlare's service you linked.


more examples in readme

More examples to illustrate rubico

Codecov Report
Patch and project coverage have no change.

Comparison is base (d42abd1) 99.60% compared to head (cca423d) 99.60%.

:exclamation: Current head cca423d differs from pull request most recent head 198125f. Consider uploading reports for the commit 198125f to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #247   +/-   ##
=======================================
  Coverage   99.60%   99.60%
=======================================
  Files         224      224
  Lines        3266     3266
=======================================
  Hits         3253     3253
  Misses         13       13

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


s2i build of jenkins does not know the installed plugins since they are only linked in the run script

I'm customizing the openshift jenkins by using the s2i build as described here: https://docs.openshift.com/container-platform/3.6/using_images/other_images/jenkins.html#jenkins-as-s2i-builder
Since this commit (
https://github.com/openshift/jenkins/commit/f7b3afb30f027d29c6bbf46f56a345e7f87e0a8c#diff-e6dcb991982488881a38bc0d22e481a0 line 208ff) the installation of plugins during s2i build does not work correctly, because at assembly time the pre-installed plugins are not yet visible for the install-plugin.sh script.
Depending on what plugins get installed, some dependent plugins with a much lower version get installed, because the do not exist for the script.
When the container is started, the links from the base installation are created, which leads to an inconsistent and unusable jeknins instance.
To assure the s2i is working correctly, the links should also/already be made from the assembly script.

On Wed, Dec 20, 2017 at 7:15 AM, Marc Brugger<EMAIL_ADDRESS>wrote:

I'm customizing the openshift jenkins by using the s2i build as described
here: https://docs.openshift.com/container-platform/3.6/using_
images/other_images/jenkins.html#jenkins-as-s2i-builder
Since this commit (
f7b3afb#diff-e6dcb991982488881a38bc0d22e481a0
https://github.com/openshift/jenkins/commit/f7b3afb30f027d29c6bbf46f56a345e7f87e0a8c#diff-e6dcb991982488881a38bc0d22e481a0)
the installation of plugins during s2i build does not work correctly,
because at assembly time the pre-installed plugins are not yet visible for
the install-plugin.sh script.
Depending on what plugins get installed, some dependent plugins with a
much lower version get installed, because the do not exist for the script.
When the container is started, the links from the base installation are
created, which leads to an inconsistent and unusable jeknins instance.

Must be the move of the linking out of the Dockerfile and into the start up
path.  Our s2i testing did not catch the nuance above @bparees.
First blush, refactor the new (re)linking bash logic into a separate .sh
file that we call from the Dockerfile.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openshift/jenkins/issues/451, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADbadP5yxEk_zSrnQTOFO36EByEXyg4eks5tCPpNgaJpZM4RITuR
.

Oh, and to be clear, we still leave the processing that does exist in install_plugins.sh to still support the upgrade scenarios that were fixed with https://github.com/openshift/jenkins/commit/f7b3afb30f027d29c6bbf46f56a345e7f87e0a8c#diff-e6dcb991982488881a38bc0d22e481a0
we just add the link back into the Dockerfile to facilitate this scenario

@bakito - would you be able to provide a precise s2i invocation that we could use as a reproduce / validator?

plugins.txt
This is the output of a s2i build with the delivered image from redhat, where the base plugins are not linked. The summary in the end lists much less plugins than actualli are installed on the jenkins once it is started. 36 plugins are installed.
s2i-log-without-base-plugins.log
This s2i log is based on an image that has the links to the base plugins done in the Dockerfile. As result, not so many transitive plugin have to be installed (8 plugins installed), since they are already available in a newer version.
s2i-log-with-links-to-base-plugins.log

thanks @bakito !
and just to be certain there are no disconnects, what is the ID and sha of the base image you used?

Name: openshift3/jenkins-2-rhel7
Summary: Provides the latest release of Red Hat Enterprise Linux 7 in a fully featured and supported base image.
Description: Jenkins is a continuous integration server
Source URL:  https://access.redhat.com/containers/#/registry.access.redhat.com/openshift3/jenkins-2-rhel7/images/v3.7.14-5
Author: Ben Parees<EMAIL_ADDRESS>Built: 7 days ago
Digest: sha256:39484e2ba9798759c173c9dc1d432b2de7559d0c0697f980d849a29f72707079
Identifier: f4981f86bde13e8065e108d0d1dabc45667b17ac2fb1acf7f261c0a698a34456

plan on having a PR with the links back in the Dockerfile, adjustments to the automated testing to ensure "base" plugins are there, as well as manual testing of the PV upgrade scenario, sometime today

ok, existing jenkins image tests are passing for me locally (both rhel and centos), and I've manually verified @bakito 's scenario with the fix.
automating the verification is a bit trickier since the existing test's starting off s2i image adds the base plugins ... I believe we'll need to add a dummy assemble script that does the base plugin file check, but I don't have that sorted out yet.
Have to run to an appointment but will complete upgrade scenario test when I get back.
Although certainly not ideal, a work around of including the contents of our base-plugins.txt into the plugins.txt consumed by s2i can do the trick.

ok, upgrade scenario with PV's still works with the link re-inserted into the dockerfile, but with the install-plugins.sh logic unchanged
sorting out additions to the tests to catch the issue, and will then craft a PR

duh .. the assemble is in the builder image ... ok, punting on including verification in automated test

duh .. the assemble is in the builder image ... ok, punting on including verification in automated test

it is, but you can provide your own assemble script as part of the inputs to the build (by providing one in your repo)
https://docs.openshift.org/latest/dev_guide/builds/build_strategies.html#override-builder-image-scripts


egraphs: Transform {u,s}widen_{low,high}+splat into splat+{u,s}extend

👋 Hey,
This PR adds an egraph optimization that transforms swiden_low+splat into splat+sextend (and similar variations).
The primary motivation for this is that I'm working on SIMD widening arithmetic instructions in the RISC-V backend, and would like to match all of those patterns with just splat+extend. That being said, It is also probably beneficial to do these transforms (at least for RISC-V) since we have a bunch of scalar instructions that sextend the result.

swiden_high et al only support doubling the width and halving the number of lanes, right?

Yeah, as far as I can tell they don't support anything else. But I did double check against the verifier, and it complains with wider types or fewer lanes.


Remove the assigner prop of issue event

Why:
Closes #20795

What's being changed (if available, include any code snippets, screenshots, or gifs):
Remove the assigner property of the timeline assigned and unassigned event. The assigner property is not included in github/rest-api-description and octokit/webhooks any more.

event schema
timeline-assigned-issue-event:
  title: Timeline Assigned Issue Event
  description: Timeline Assigned Issue Event
  type: object
  properties:
    id:
      type: integer
    node_id:
      type: string
    url:
      type: string
    actor:
      "$ref": "#/components/schemas/simple-user"
    event:
      type: string
    commit_id:
      type:
      - string
      - 'null'
    commit_url:
      type:
      - string
      - 'null'
    created_at:
      type: string
    performed_via_github_app:
      anyOf:
      - type: 'null'
      - "$ref": "#/components/schemas/integration"
    assignee:
      "$ref": "#/components/schemas/simple-user"

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@yanyongyu Thanks so much for opening a PR! I'll get this triaged for review :zap:


Media Thumbnail controller fallback image (coffee icon) dont work correctly

Broken Images appears instead of fallback Icon

A bit more information would be useable :P

@danrot screenshot added

How did that happen?

@danrot did test this behaviour it only happens on chrome.
It seems when the http status code is 404 which I set when there is no thumbnail image available for this file(coffee icon fallback) the chrome dont render the fallback image. Should we change the HTTP Status code or what should we do?

How is that fallback rendered? It's really strange that this only happens on chrome. And if not totally necessary I would not change the http status code...

If the uploaded file is not an image or cant be converted to an image the coffee icon image is returned with the status 404. If you open the image url directly in a new tab the image is loaded.

I don't think that returning the coffee icon with a status code of 404 is a good idea... Chromium's behaviour is correct IMHO.
Returning the coffee icon with every 404 response also means that it will be transferred every time. I would just return a 404 without the coffee icon, and change the path on the client side. Then this would work correctly and the image would only be transferred once.

thats a good idea. I found out that the browser through an event when the image can't be loaded (404).
so we can handle the coffee icon fallback in the frontend.
$('img').bind('error', function() {


drivers: hwinfo: add driver support for Atmel SAM0 device ID

Add driver support for Atmel SAM0 device ID, which is 16-bytes long.
The device ID can simply be read from memory at a known location, but
the location is only described in the data sheet, not in ASF.
For SAMD2x it's 0x0080A00C, 0x0080A040, 0x0080A044 & 0x0080A048.
For SAMD5x it's 0x008061FC, 0x00806010, 0x00806014 & 0x00806018.
This adds a new property to the device tree to define the device ID registers for this SoC family.

Codecov Report

Merging #14908 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #14908      +/-   ##
==========================================
- Coverage   52.61%   52.57%   -0.04%
==========================================
  Files         309      309
  Lines       45113    45113
  Branches    10445    10445
==========================================
- Hits        23734    23718      -16
- Misses      16569    16581      +12
- Partials     4810     4814       +4

Impacted Files
Coverage Δ

subsys/testsuite/ztest/include/ztest_assert.h
38.88% <0%> (-38.89%)
:arrow_down:

subsys/testsuite/ztest/src/ztest.c
65.28% <0%> (-7.44%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 46d9d3f...732ff4f. Read the comment docs.


Bind function to class instance in middleware

For #9096
Haven't found tests for this specific class.

[X] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[X] Title summarizes what is changing
- [ ] Has a news entry file (remember to thank yourself!)
[X] Appropriate comments and documentation strings in the code
- [ ] Has sufficient logging.
- [ ] Has telemetry for enhancements.
- [ ] Unit tests & system/integration tests are added/updated
- [ ] Test plan is updated as appropriate
- [ ] package-lock.json has been regenerated by running npm install (if dependencies have changed)
- [ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #9109 into master will decrease coverage by 0.69%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #9109     +/-   ##
=========================================
- Coverage   57.11%   56.42%   -0.7%
=========================================
  Files         551      551
  Lines       29144    28775    -369
  Branches     4404     4404
=========================================
- Hits        16646    16235    -411
- Misses      11592    11633     +41
- Partials      906      907      +1

Impacted Files
Coverage Δ

...ivation/languageServer/languageClientMiddleware.ts
8.1% <100%> (+1.25%)
:arrow_up:

...nt/datascience/jupyter/invalidNotebookFileError.ts
0% <0%> (-60%)
:arrow_down:

src/client/formatters/serviceRegistry.ts
50% <0%> (-50%)
:arrow_down:

src/datascience-ui/native-editor/redux/store.ts
50% <0%> (-50%)
:arrow_down:

src/datascience-ui/history-react/redux/store.ts
50% <0%> (-50%)
:arrow_down:

src/datascience-ui/react-common/logger.ts
0% <0%> (-40%)
:arrow_down:

...cience/jupyter/jupyterDebuggerNotInstalledError.ts
33.33% <0%> (-33.34%)
:arrow_down:

src/datascience-ui/react-common/locReactSide.ts
16.66% <0%> (-33.34%)
:arrow_down:

...jupyter/kernels/jupyterKernelPromiseFailedError.ts
0% <0%> (-33.34%)
:arrow_down:

...ent/datascience/jupyter/jupyterWaitForIdleError.ts
0% <0%> (-33.34%)
:arrow_down:

... and 45 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 844a423...8f827f3. Read the comment docs.


controller.storage.users returns undefined

Retrieving data from the "built in storage object" of Botkit does not seem to work when using either a mongoDB connection or just default config.
controller.storage.users always returns cannot read property users of undefined
I'm currently connecting to MongoDB Atlas which confirmed I have connected to my MongoDB, so that shouldn't be the problem. Using the application without a MONGO_URI defined returns the same response.
All docs and examples refer to this, but I can't find any working example out there which work for v4.

controller.storage.users only applies to Botkit v0.7 and below.
v4 does not provide a built in mechanism for storing your user info.

Sorry for the rookie questions, but what's the thing with the Mongo URI configuration etc if you can't so anything with it? AFAIK there isn't anything documented about actually using the storage.
How would you replicate for example the tasklist example of the old slack-starter-kit?

The Mongo is used for the underlying Bot Framework storage system, which is used to store conversation state.  In the future, we may expand that (as Bot Framework adds features), but in general it is my recommendation that you use your own database accessor.
This is complete pseudo-code, but it would be something like this:
module.exports = function(controller) {

 const my_db = new database(uri);

controller.hears('add todo', 'message', async(bot, message) => {
    let task = message.text;
     await my_db.insert(message.user, task);
});

}


refactor(AutoOperate): extracting common interfaces for AutoOperate related components (GitHub #166)

…

Codecov Report
Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (9751bd8) 91.03% compared to head (db3e6c0) 91.01%.
Report is 3 commits behind head on dev.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev     #167      +/-   ##
============================================
- Coverage     91.03%   91.01%   -0.03%
  Complexity     1382     1382
============================================
  Files           136      136
  Lines          3314     3315       +1
  Branches        314      315       +1
============================================
  Hits           3017     3017
  Misses          164      164
- Partials        133      134       +1

Files
Coverage Δ

...re/support/aop/MethodResultAutoOperateSupport.java
81.25% <ø> (ø)

...sion/spring/DefaultCrane4jSpringConfiguration.java
94.91% <100.00%> (ø)

...j/spring/boot/config/Crane4jAutoConfiguration.java
93.57% <100.00%> (ø)

...j/core/support/aop/AutoOperateAnnotatedMethod.java
92.30% <92.30%> (ø)

.../support/aop/MethodArgumentAutoOperateSupport.java
88.63% <0.00%> (-2.07%)
:arrow_down:

...ethodBasedAutoOperateAnnotatedElementResolver.java
76.74% <76.74%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


add methods that make it easy to make a mergeable store from java.

for example: MemcacheStore.mergeableFrom
default values and implicit parameters are hard to use from java.

see #624


Add support for syncing Windows binaries

What this PR does / why we need it:
Modifies scripts/download_k8s_binaries to sync Windows binaries as well as Linux
How can this PR be verified?
Run ./scripts/download_k8s_binaries 1.13.4, see that it updates for both Windows and Linux packages. Requires bash >= 4.3.
Is there any change in kubo-deployment?
Nope
Is there any change in kubo-ci?
I don't think any is required
Does this affect upgrade, or is there any migration required?
No
Release note:

NONE

cc @tvs @tenczar

As part of testing this we used it to facilitate the 1.13.4 bump and merged through https://github.com/cloudfoundry-incubator/kubo-release/pull/313.
Thanks!


fix get_browsers. w3-table-all --> ws-table-all

settings.BROWSERS_STATS_PAGE html changed.

fixes #112

also fixes #113

Why is it still not merged?

This PR needs to be merged immediately. 🥲

same PR https://github.com/hellysmile/fake-useragent/pull/125

Hello, why this pretty old PR is not being accepted/rejected if it fixes a current bug (present in the pypi packaged version)? Is this project abandonned?

Fixed by: https://github.com/hellysmile/fake-useragent/pull/136


OpenType fonts  are reported as format type "CFF"

I expected the format type to be "OpenType" but "CFF" was reported. I see that the font has a CFF table, but I don't understand why that would define its format when TrueType fonts are reported as TrueType.

Yes, there's is actually a *cff format that also gets reported as "CFF" by the  underlying FreeType library.
OpenType *otf fonts theoretically have two variants that differ structurally CFF with Type1 outlines and TT with TrueType outlines, but I've never seen examples of the latter.
I'll have a look and see if there's some way of distinguishing simple CFF and OpenType CFF.

Have changes as suggested. OpenType CFF now returns format OpenType. These fonts usually have extension *.otf.
Fix released with Font::FreeType  0.5.5

Thanks!


Currently Returning 0 Results for Today's Standup Query

https://github.com/NCATSTranslator/testing/blob/main/ars-requests/not-none/pathway.json

This is now returning no results

Was it ever returning results? I thought we had determined that is was correctly returning no results.

I think we were somehow returning results... checking the ARS.
The BTE issue https://github.com/biothings/biothings_explorer/issues/176 should allow us to get results.  I'm not sure if its deployed yet, but when it is, we might need to update our internal registry?

Turns out that when we were returning results it was from a different (buggy) endpoint, so those were not real.
We should also be able to get Gene/Pathway from the CAM-KP, but it's contingent on this issue: https://github.com/NCATS-Tangerine/cam-pipeline/issues/59

WIth the BTE fix, we are now returning results for this query


How to use this sdk to do client_credentials authorization

How to do the process described in this page ?
https://developer.okta.com/docs/guides/implement-grant-type/clientcreds/main/

@wjlabcwjl Could you please, specify what exactly you want to achieve with our sdk? This SDK is used to interact with Okta management API. If you need to manipulate with tokens, we have okta-jwt-verifier for token validation, for example.

@wjlabcwjl Could you please, specify what exactly you want to achieve with our sdk? This SDK is used to interact with Okta management API. If you need to manipulate with tokens, we have okta-jwt-verifier for token validation, for example.

I want to request a token with client credentials grant type flow, I'm looking for a python helper which can do the base64 encoding and make a request with basic authorization header then return me the token.

As a python helper I can suggest write custom function, something like the following:
import requests
import base64

url = f"https://{yourOktaDomain}/oauth2/default/v1/token"
auth = f'{clientId}:{clientSecret}'
base64_encoded = base64.b64encode(auth.encode('ascii')).decode('ascii')
headers = {'Accept': 'application/json',
           'Content-Type': 'application/x-www-form-urlencoded',
           'Authorization': f'Basic {base64_encoded}'}
data = 'grant_type=client_credentials&scope=customScope'

response = requests.post(url, headers=headers, data=data)
print(response.text)

As a python helper I can suggest write custom function, something like the following:
import requests
import base64

url = f"https://{yourOktaDomain}/oauth2/default/v1/token"
auth = f'{clientId}:{clientSecret}'
base64_encoded = base64.b64encode(auth.encode('ascii')).decode('ascii')
headers = {'Accept': 'application/json',
           'Content-Type': 'application/x-www-form-urlencoded',
           'Authorization': f'Basic {base64_encoded}'}
data = 'grant_type=client_credentials&scope=customScope'

response = requests.post(url, headers=headers, data=data)
print(response.text)

Does it work for you?

This is what I'm doing and it works, just want to avoid this little code, I thought your sdk could do this, I see this method : https://github.com/okta/okta-sdk-python/blob/master/okta/oauth.py#L30
Well, if this is not the purpose of your sdk, you can close my issue, thanks for your help.

And yes, you can access these internals if you want:
from okta.client import Client

client = Client()

access_token, error = await client._request_executor._oauth.get_access_token()

Then I'm closing this issue as resolved, feel free to reopen it or create a new one if your problem exists still.

From what I see, this client only supports API tokens and PrivateKey authorizations: https://github.com/okta/okta-sdk-python/blob/46184663baccb926dc14ab964dfd9406d375de0a/okta/config/config_validator.py#L43
So to make explicit, if we want to make a request with a client credentials access token, this client does not support our use case?
Is there a good reason to not support bearer authorization?

@erichulburd Few years ago API didn't support Bearer tokens https://devforum.okta.com/t/using-oauth2-bearer-token-to-access-okta-apis/845/9
I think, we can add support for bearer authorization. I'll add this feature to our backlog.

@erichulburd SDK v2.2.0 has been released, now you can pass your access token using "Bearer" authorization mode

Just wanted to say, it took me a while because the link between the documentation and the api documentation, seems to not exist...
I mean I appreciate the REST endpoints...but I was hoping for some native python examples USING the SDK for various grant flows for a native python app trying to talk to a protected resource.
My use case specifically is that I am looking to protect a new API and we were hoping to use Okta to do it. The new API is a Spring Boot application, and the Java implementation is really straight forward, thanks to this little tutorial here: https://developer.okta.com/blog/2020/08/07/spring-boot-remote-vs-local-tokens
This Python app is not your traditional, its an app that will be run at scale for automation...I was hoping to see how easy it was to adopt PKCE WITHOUT rest endpoints...but after reading around, it was suggested that perhaps client credentials is the way to go.
However for the Python side, I struggled for like a few days trying to see:

what the THEORY of OAuth was
what Okta offered AND
how this SDK supported what Okta offered and matched up against the theory

I ended up throwing in the towel of trying to make sense of the documentation and started reading the unit tests (which I am not sure if that is good or bad, usually if I find myself digging into third party code...its probably a bad sign of something) to see what classes I could use, what are my entrypoints into the Okta ecosystem VIA the Okta SDK lib and figure out how one might hook myself up to get a single JWT...
And so, here are the fruits of my labour (for better or for worse, feel free to let me know if I did anything horrendous or heretical, I am merely fumbling my way through this library):
from okta.oauth import OAuth # this is my oauth client
from okta.request_executor import RequestExecutor # unsure if this is good practice, I was not clear what this class did or what it was for, but apparently its needed?
import json

with open('public_private_pair.key', 'r') as f: # here I am extracting my JWK generated from: https://mkjwk.org/ as per the Okta documentation
    public_private_key = json.loads(f.read())

config = {
    'client': { # apparently you need a config with a client attributed nested inside? Not 100% sure why you need this, but the code seems to imply this
        'orgUrl': org_url # this is your okta domain URL
        'authorizationMode': None # I do not know what this is, I could also use `PrivateKey` but I was unsure what this actually did in Python land
        'clientId': client_id # this is your client id, defined in the Okta service app (once you have created it in the okta admin page)
        'scopes': [your_custom_api_scope] # this is the scope enabled on the Okta service app, if you are spinning up a protected resource, it must match the scope provided on the Okta admin page
        'rateLimit': {} # after a lot of tinkering, it seems it does not support numbers, and it only supports a dict (was not aware of this), subbing in a blank dict seems to let the calls pass
        'privateKey': public_private_key # you could also use an x.509 certificate as well here, the code seems to support both
    }
}

request_executor = RequestExecutor(config=config, cache={}) # not sure what the cache is for, I just subbed it out with a blank dict
oauth_client = OAuth(request_executor=request_executor, config=config) # not entirely sure why you need to provide the config twice, when the config is in the request_executor object, but anyways I subbed it in again
jwt = oauth_client.get_JWT() # this is the CLIMACTIC moment that my several days of tinkering led up to, I could finally get the damned JWT JUST to forward that over to my protected API resource so that I can verify the signature of it
print(jwt) # haha all I wanted was this little JWT and it was so much pain to reverse engineer what my inputs were to get to this point

Please let me know, if I have done anything heretical in my set up, but this ended up working for me, and I could verify the JWT that came back out via: https://jwt.io/ and just subbed in my dummy private and public keys.
Or feel free to let me know that I am blind and forward the documentation over my way, I could not find a lot of documentation surrounding this workflow nor examples. Most of the examples seem to be about receiving a JWT and just verifying it...which is nice...but I want to generate one to have it verified.

@SquireOfSoftware Sorry to hear that you have had a lot of pain. Hope, I can help you to make it clear.
So, first of all, this SDK is intended to work with management APIs, i.e. it is a client to make easy and consistent interaction with APIs which are described within openapi spec https://github.com/okta/okta-management-openapi-spec. Generating and validating JWTs is not a responsibility of this SDK, although you can generate a valid token as you described in the sample above, but I can suggest a bit easier code sample:
import asyncio
from okta.client import Client as OktaClient

private_key = load_private_key() # here you can use your approach or anything else to get private key

config = {
    'orgUrl': 'https://<YOUR_ORG_URL>.okta.com',
    'authorizationMode': 'PrivateKey',
    'clientId': '<CLIENT_ID>',
    'scopes': ['<YOUR_CUSTOM_API_SCOPE>'],
    'privateKey': f'{private_key}'
}

async def main():
    client = OktaClient(config)
    print(client._request_executor._oauth.get_JWT()) # generate and print access token

loop = asyncio.get_event_loop()
loop.run_until_complete(main())

For local token validation we have another library: https://github.com/okta/okta-jwt-verifier-python, which can be installed via pip too: pip install okta-jwt-verifier
Let me know if it helps. Do not hesitate to ask more and thanks for using OKTA!